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