[PATCH] Use function attribute "arm-restrict-it"

Akira Hatanaka ahatanak at gmail.com
Mon Jun 22 11:43:46 PDT 2015


================
Comment at: lib/Target/ARM/ARMSubtarget.cpp:320
@@ -330,1 +319,3 @@
 
+bool ARMSubtarget::restrictIT(const Function &F) const {
+  // The default behavior is to disallow generation of deprecated IT blocks
----------------
compnerd wrote:
> Bikeshedding: I think that restrictedIT might be a better name.  IIRC, the ARM ARM refers to this as Restricted IT Blocks.
I'm not sure whether restrictIT is a good name or not, but if we are going to rename it, should we use a verb like "useRestrictedIT" as llvm's coding style guide recommends?

http://reviews.llvm.org/D10416

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list