[PATCH] D102908: [lld][ELF][SPARC] Support TLS IE relocations

[Jessica Clarke via Phabricator via llvm-commits]

jrtc27 added inline comments.


================
Comment at: lld/ELF/Driver.cpp:929
   return m == EM_AARCH64 || m == EM_AMDGPU || m == EM_HEXAGON || m == EM_PPC ||
-         m == EM_PPC64 || m == EM_RISCV || m == EM_X86_64;
+         m == EM_PPC64 || m == EM_RISCV || m == EM_SPARCV9 || m == EM_X86_64;
 }
----------------
LemonBoy wrote:
> jrtc27 wrote:
> > This should probably be its own commit, it's unrelated to TLS.
> This _is_ related to TLS as, without this change, the `R_SPARC_TLS_TPOFF64` would end up in REL form and the resulting program be rejected by `ld.so`.
Why is `R_SPARC_TLS_TPOFF64` special? Surely that's true of any dynamic relocation.


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

https://reviews.llvm.org/D102908