[PATCH] D118044: [ARM] Undeprecate complex IT blocks

David Spickett via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 24 08:47:36 PST 2022


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



More information about the llvm-commits mailing list