[PATCH] D31751: [LLD][ELF] Tidy up handleARMTlsRelocation [NFC]

Rafael Ávila de Espíndola via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 6 15:09:20 PDT 2017


rafael added inline comments.


================
Comment at: ELF/Relocations.cpp:160
+  // we know the offset into the TLS block at static link time.
+  bool NeedDynId = Body.isPreemptible() || Config->Pic;
+  bool NeedDynOff = Body.isPreemptible();
----------------
Why Config->Pic? Is it not always 1 for position independent executables?


https://reviews.llvm.org/D31751





More information about the llvm-commits mailing list