DavidSpickett added inline comments.

================
Comment at: llvm/lib/Target/ARM/AsmParser/ARMAsmParser.cpp:10977
-        Warning(IDLoc, "deprecated instruction in IT block");
-      }
     }
----------------
MarkMurrayARM wrote:
> DavidSpickett wrote:
> > 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.
> No - in effect, deprecation is gone entirely. There is nothing to warn about. 
> You can ask the compiler not to do it, but if you craft complex IT blocks by 
> hand, that is up to you.
Got it, agreed.


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