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



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

Reply via email to