[llvm] [RISCV] Support Global Dynamic TLSDESC in the RISC-V backend (PR #66915)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 12:41:25 PST 2024


================
@@ -56,3 +56,7 @@ ELF_RELOC(R_RISCV_IRELATIVE,         58)
 ELF_RELOC(R_RISCV_PLT32,             59)
 ELF_RELOC(R_RISCV_SET_ULEB128,       60)
----------------
MaskRay wrote:

> Related: do you think its worth testing that they're not there?

I've made a comment on the test "I think we should use llvm-objdump -d -r to check relocations."
If you do that, `-NEXT` is sufficient to ensure that there is no `R_RISCV_RELAX` :)

With my experience on https://github.com/llvm/llvm-project/pull/79099 , I think two `R_RISCV_RELAX` are sufficient. I probably need to follow up on the spec.

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


More information about the llvm-commits mailing list