LoongArch: Fixed R_LARCH_[32/64]_PCREL generation bug

The enum BFD_RELOC_[32/64] was mistakenly used in the macro instead
of the relocation in fixp. This can cause the second relocation
of a pair to be deleted when -mthin-add-sub is enabled. Apply the
correct macro to fix this.

Also sets the initial value of -mthin-add-sub.

diff --git a/gas/config/tc-loongarch.h b/gas/config/tc-loongarch.h
index 20434ac8a0153cb1a08da362e960c1975c17d53f..64d95e44277af7b1a01431fe8345ea684bdadf71 100644 (file)
--- a/gas/config/tc-loongarch.h
+++ b/gas/config/tc-loongarch.h
@@ -79,7 +79,8 @@ extern bool loongarch_frag_align_code (int, int);
    SEC_CODE, we generate 32/64_PCREL.  */
 #define TC_FORCE_RELOCATION_SUB_LOCAL(FIX, SEG) \
   (!(LARCH_opts.thin_add_sub \
-     && (BFD_RELOC_32 || BFD_RELOC_64) \
+     && ((FIX)->fx_r_type == BFD_RELOC_32 \
+        ||(FIX)->fx_r_type == BFD_RELOC_64) \
      && (!LARCH_opts.relax \
        || S_GET_VALUE (FIX->fx_subsy) \
           == FIX->fx_frag->fr_address + FIX->fx_where \

diff --git a/gas/testsuite/gas/loongarch/no_thin_add_sub_8_16.d b/gas/testsuite/gas/loongarch/no_thin_add_sub_8_16.d
new file mode 100644 (file)
index 0000000..334d174
--- /dev/null
+++ b/gas/testsuite/gas/loongarch/no_thin_add_sub_8_16.d
@@ -0,0 +1,12 @@
+#as: -mthin-add-sub
+#readelf: -rW
+#skip: loongarch32-*-*
+
+Relocation section '.rela.text' at offset 0x108 contains 6 entries:
+.*
+0+0  000000060000002f R_LARCH_ADD8           0+0 global_a \+ 0
+0+0  0000000400000034 R_LARCH_SUB8           0+0 L0\^A \+ 0
+0+1  0000000600000030 R_LARCH_ADD16          0+0 global_a \+ 0
+0+1  0000000500000035 R_LARCH_SUB16          0+1 L0\^A \+ 0
+0+3  0000000600000063 R_LARCH_32_PCREL       0+0 global_a \+ 0
+0+7  000000060000006d R_LARCH_64_PCREL       0+0 global_a \+ 0

diff --git a/gas/testsuite/gas/loongarch/no_thin_add_sub_8_16.s b/gas/testsuite/gas/loongarch/no_thin_add_sub_8_16.s
new file mode 100644 (file)
index 0000000..68f3655
--- /dev/null
+++ b/gas/testsuite/gas/loongarch/no_thin_add_sub_8_16.s
@@ -0,0 +1,6 @@
+    .text
+.L1:
+    .byte   global_a - .
+    .2byte  global_a - .
+    .4byte  global_a - .
+    .8byte  global_a - .

diff --git a/opcodes/loongarch-opc.c b/opcodes/loongarch-opc.c
index 6bd436fa1fe6cb968799f6f8a0c9a370e3ad22fa..0ba745f7d0aa1a7ddeca7270a786a5e393356173 100644 (file)
--- a/opcodes/loongarch-opc.c
+++ b/opcodes/loongarch-opc.c
@@ -24,7 +24,8 @@
 
 struct loongarch_ASEs_option LARCH_opts =
 {
-  .relax = 1
+  .relax = 1,
+  .thin_add_sub = 0
 };
 
 size_t