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

2017-03-16 Thread Oliver Stannard via Phabricator via cfe-commits
olista01 added a comment. My patch to the ACLE has now been accepted, so it will be in the next release. This is the wording: __ARM_ROPI is defined to 1 if the translation unit is being compiled in read-only position independent mode. In this mode, all read-only data and functions are at a

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

2016-12-14 Thread Oliver Stannard via Phabricator via cfe-commits
olista01 added a comment. I've proposed a patch to the ACLE to add these macros, but it's stalled in review. I've just given it a ping, I'll try to get an answer soon. Repository: rL LLVM https://reviews.llvm.org/D23610 ___ cfe-commits mailing

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

2016-12-12 Thread Renato Golin via Phabricator via cfe-commits
rengolin added a comment. Was this abandoned? 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

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

2016-08-17 Thread Tim Northover via cfe-commits
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

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

2016-08-17 Thread Oliver Stannard via cfe-commits
olista01 created this revision. olista01 added reviewers: rengolin, t.p.northover. olista01 added a subscriber: cfe-commits. olista01 set the repository for this revision to rL LLVM. Herald added subscribers: samparker, rengolin, aemerson. This adds pre-defined macros to test for code being