[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