[PATCH] D103815: [LLD][ELF] Fix PT_LOAD program header creation for NO_LOAD sections
Fangrui Song via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 15 13:14:33 PDT 2021
MaskRay accepted this revision.
MaskRay added a comment.
This revision is now accepted and ready to land.
LGTM, but the subject and description need to be updated.
We drop the special case that `(NOLOAD)` is not placed in a PT_LOAD. The new logic aligns with modern GNU ld.
If merging PT_LOAD will add more complexity to `sameLMARegion`, I'd suggest we don't do that.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103815/new/
https://reviews.llvm.org/D103815
More information about the llvm-commits
mailing list