[clang] [llvm] [X86] Add Support for X86 TLSDESC Relocations (PR #83136)

[Paul Kirth via llvm-commits]

https://github.com/ilovepi commented:

I assume that's what you're doing in the place I asked for a comment is trying to avoid redundantly generating TLSDESC accesses for _TLS_MODULE_BASE_.  I see that we don't generate multiple  accesses for _TLS_MODULE_BASE_ in the test cases, but I thought that was only safe to do when an earlier access dominates a later access.  

Is my understanding here correct, or is there an interaction with  https://github.com/llvm/llvm-project/blob/04bbbba271ebe4c2421f34a4fbf34c328df9f111/llvm/lib/Target/X86/X86InstrInfo.cpp#L10245 ?

https://github.com/llvm/llvm-project/pull/83136