[PATCH] D23610: [ARM] Add pre-defined macros for ROPI, RWPI and FPIC

Tim Northover via cfe-commits cfe-commits at lists.llvm.org
Wed Aug 17 12:16:56 PDT 2016


t.p.northover added inline comments.

================
Comment at: lib/Basic/Targets.cpp:5316
@@ +5315,3 @@
+    if (Opts.PICLevel)
+      Builder.defineMacro("__APCS_FPIC", "1");
+    if (Opts.ROPI)
----------------
These names are pretty bad from a modern perspective (with APCS deprecated and almost entirely forgotten outside iOS). It's the kind of thing it would be good for the ACLE to cover, would they be likely to consider it?


Repository:
  rL LLVM

https://reviews.llvm.org/D23610





More information about the cfe-commits mailing list