Re: [PATCH] D23196: [ARM] Command-line options for embedded position-independent code

2016-08-08 Thread Oliver Stannard via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL278016: [ARM] Command-line options for embedded position-independent code (authored by olista01). Changed prior to commit: https://reviews.llvm.org/D23196?vs=66921&id=67176#toc Repository: rL LLVM h

Re: [PATCH] D23196: [ARM] Command-line options for embedded position-independent code

2016-08-05 Thread Renato Golin via cfe-commits
rengolin accepted this revision. rengolin added a comment. This revision is now accepted and ready to land. Hi Oliver, The driver part looks good to me, thanks! I haven't looked at the LLVM side of the patch. The other reviews probably know more than I do about that, anyway. :) cheers, --renat

[PATCH] D23196: [ARM] Command-line options for embedded position-independent code

2016-08-05 Thread Oliver Stannard via cfe-commits
olista01 created this revision. olista01 added reviewers: t.p.northover, rengolin, srhines, weimingz. olista01 added a subscriber: cfe-commits. olista01 set the repository for this revision to rL LLVM. Herald added subscribers: samparker, rengolin, aemerson. This patch (with the corresponding ARM