[PATCH] D50133: [LLD] Do not overwrite LMAOffset of PT_LOAD header

George Rimar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 2 03:32:59 PDT 2018


grimar accepted this revision.
grimar added a comment.
This revision is now accepted and ready to land.

LGTM, thanks! I'll commit this for you and land a little refactoring change on top I think.

(btw, I think you can request for commit access now if you want it).



================
Comment at: ELF/LinkerScript.cpp:758
+
+static OutputSection *findFirstSection(PhdrEntry *Load) {
+  for (OutputSection *Sec : OutputSections)
----------------
It is taken from LinkerScript.cpp I guess. I think it is ok to have a copy for now since this helper is little.


https://reviews.llvm.org/D50133





More information about the llvm-commits mailing list