DavidSpickett added a comment.

Please define what "complex" is, in the commit message and (if it doesn't make 
the description a whole paragraph) the command line help.

Given that we're flipping a default for v8 thumb targets does this warrant a 
release note?



================
Comment at: llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp:10977
-        Warning(IDLoc, "deprecated instruction in IT block");
-      }
     }
----------------
Is this warning something you'd still want if you opted into restricting IT 
blocks? Or can we say that because there has/will not be any core produced that 
follows the deprecation, you would never need the warning.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D118044/new/

https://reviews.llvm.org/D118044

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to