[PATCH] D24298: [ELF] Linkerscript: Implement LOADADDR
Eugene Leviant via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 13 02:29:36 PDT 2016
evgeny777 added inline comments.
================
Comment at: ELF/LinkerScript.cpp:431
@@ -416,3 +430,3 @@
if ((Sec->getFlags() & SHF_TLS) && Sec->getType() == SHT_NOBITS) {
uintX_t TVA = Dot + ThreadBssOffset;
----------------
This wasn't needed, because setLMAOffset is called in SetAddrs(). See lines 399 - 403
================
Comment at: ELF/Writer.cpp:1003-1004
@@ -992,4 +1002,4 @@
uintX_t NewFlags = Sec->getPhdrFlags();
- if (Script<ELFT>::X->getLma(Sec->getName()) || Flags != NewFlags) {
+ if (HasLMA || Flags != NewFlags) {
Load = AddHdr(PT_LOAD, NewFlags);
Flags = NewFlags;
----------------
See lines 987-995
https://reviews.llvm.org/D24298
More information about the llvm-commits
mailing list