[PATCH] D62577: [ELF] Support Local Dynamic style TLSDESC for x86-64

[George Rimar via Phabricator via llvm-commits]

grimar added inline comments.


================
Comment at: ELF/Writer.cpp:1626
+                         /*Section=*/nullptr});
+    Symtab->TlsModuleBase = S;
+  }
----------------
Should this assignment be under `if (S && S->isUndefined())` check?

I am not sure how much is real (I guess not at all honestly). but if user defines his own `_TLS_MODULE_BASE_` symbol, then
with the current code `getTlsTpOffset` would return 0 for it. It is probably not user would expect.


Repository:
  rLLD LLVM Linker

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

https://reviews.llvm.org/D62577