[PATCH] D37580: Add Position Independent Pages (PIP) relocation model

Stephen Crane via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 31 11:40:30 PDT 2017


rinon added a comment.

Thanks for taking a look at this!

It looks like everything but the first revision got sent to llvm-commits (http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20171030/498777.html). Since I've rebased all the patches, the diffs have also been sent to the list. If this is insufficient I'm happy to abandon and reopen revisions; I just don't want to make the conversations harder to follow on phabricator since there are already comments.

Good pooint, I'll link to the RFC to the summary, since the first comment I wrote on each patch will get hidden after a long discussion:

In https://reviews.llvm.org/D37580#863793, @rinon wrote:

> This patch set (https://reviews.llvm.org/D37580, https://reviews.llvm.org/D37581, https://reviews.llvm.org/D37582, https://reviews.llvm.org/D37583, https://reviews.llvm.org/D37584, https://reviews.llvm.org/D37585, https://reviews.llvm.org/D37586, https://reviews.llvm.org/D37587) is a first draft of the pagerando implementation described in http://lists.llvm.org/pipermail/llvm-dev/2017-June/113794.html.



https://reviews.llvm.org/D37580





More information about the llvm-commits mailing list