[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