[PATCH] D38361: [ELF] Stop setting output section size early

James Henderson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 04:57:21 PDT 2017


jhenderson added a comment.

In https://reviews.llvm.org/D38361#891842, @jhenderson wrote:

> In https://reviews.llvm.org/D38361#890717, @peter.smith wrote:
>
> > I recommend splitting out the SHF_LINK_ORDER change into a separate patch if it is unrelated.
>
>
> It's not entirely unrelated, but it could be a separate prerequisite patch. I'll look at splitting it off.


I've created https://reviews.llvm.org/D38687 for this. It could be a standalone change, but the motivation is less important in that situation. @ruiu, if you prefer that I do it as part of this change directly, I'm happy to do so.

I'm going to leave the corresponding changes in this diff, unless the other review is committed.


https://reviews.llvm.org/D38361





More information about the llvm-commits mailing list