[lld] [llvm] MIPS: Use pcrel|sdata4 for eh_frame (PR #91291)

YunQiang Su via llvm-commits llvm-commits at lists.llvm.org
Mon May 6 21:06:06 PDT 2024


================
@@ -343,7 +343,7 @@ void MCObjectFileInfo::initELFMCObjectFileInfo(const Triple &T, bool Large) {
   case Triple::mips64el:
     // We cannot use DW_EH_PE_sdata8 for the large PositionIndependent case
     // since there is no R_MIPS_PC64 relocation (only a 32-bit version).
-    if (PositionIndependent && !Large)
+    if (PositionIndependent)
----------------
wzssyqa wrote:

Yes. even with `Large` we still use sdata4, instead of sdata8.

```
For N64, maybe we should use sdata8, while GNU ld doesn't support it well, and in fact sdata4 is enough now.
```
Maybe, I will need to make it more clear.

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


More information about the llvm-commits mailing list