[PATCH] D57821: [LLD][ELF] - Set DF_STATIC_TLS flag for X64 target

Peter Collingbourne via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 6 11:07:48 PST 2019


pcc added inline comments.


================
Comment at: ELF/Arch/X86_64.cpp:84
                                  const uint8_t *Loc) const {
+  if (Type == R_X86_64_GOTTPOFF)
+    Config->HasStaticTlsModel = true;
----------------
Should this be done in `scanRelocs`? Then I believe that `HasStaticTlsModel` would not need to be atomic.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D57821/new/

https://reviews.llvm.org/D57821





More information about the llvm-commits mailing list