riscv
/
binutils-gdb
1
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity
Browse Source

MIPS/GAS: Treat local jump relocs the same no matter if REL or RELA 

Do not convert jump relocs against local MIPS16 or microMIPS symbols to
refer to a section symbol instead even on RELA targets, as it makes it
impossible for the linker to make a JAL to JALX conversion based on ISA
symbol annotation, breaking regular and compressed MIPS interlinking.

	gas/
	* config/tc-mips.c (mips_fix_adjustable): Also return 0 for
	jump relocations against MIPS16 or microMIPS symbols on RELA
	targets.
	* testsuite/gas/mips/jalx-local.d: New test.
	* testsuite/gas/mips/jalx-local-n32.d: New test.
	* testsuite/gas/mips/jalx-local-n64.d: New test.
	* testsuite/gas/mips/jalx-local.s: New test source.
	* testsuite/gas/mips/mips.exp: Run the new tests.

	ld/
	* testsuite/ld-mips-elf/jalx-local.d: New test.
	* testsuite/ld-mips-elf/jalx-local-n32.d: New test.
	* testsuite/ld-mips-elf/jalx-local-n64.d: New test.
	* testsuite/ld-mips-elf/mips-elf.exp: Run the new tests.
binutils-2_27-branch
Maciej W. Rozycki 10 years ago
parent
4512dafa50
commit
44d3da2338
12 changed files with 195 additions and 7 deletions
Whitespace
Show all changes Ignore whitespace when comparing lines Ignore changes in amount of whitespace Ignore changes in whitespace at EOL
Split View
Diff Options
Show Stats Download Patch File Download Diff File
  1. 11
      gas/ChangeLog
  2. 13
      gas/config/tc-mips.c
  3. 29
      gas/testsuite/gas/mips/jalx-local-n32.d
  4. 37
      gas/testsuite/gas/mips/jalx-local-n64.d
  5. 28
      gas/testsuite/gas/mips/jalx-local.d
  6. 33
      gas/testsuite/gas/mips/jalx-local.s
  7. 3
      gas/testsuite/gas/mips/mips.exp
  8. 7
      ld/ChangeLog
  9. 6
      ld/testsuite/ld-mips-elf/jalx-local-n32.d
  10. 6
      ld/testsuite/ld-mips-elf/jalx-local-n64.d
  11. 26
      ld/testsuite/ld-mips-elf/jalx-local.d
  12. 3
      ld/testsuite/ld-mips-elf/mips-elf.exp

11
gas/ChangeLog
View File

@ -1,3 +1,14 @@
2016-05-24 Maciej W. Rozycki <macro@imgtec.com>
* config/tc-mips.c (mips_fix_adjustable): Also return 0 for
jump relocations against MIPS16 or microMIPS symbols on RELA
targets.
* testsuite/gas/mips/jalx-local.d: New test.
* testsuite/gas/mips/jalx-local-n32.d: New test.
* testsuite/gas/mips/jalx-local-n64.d: New test.
* testsuite/gas/mips/jalx-local.s: New test source.
* testsuite/gas/mips/mips.exp: Run the new tests.
2016-05-24 Maciej W. Rozycki <macro@imgtec.com>
* config/tc-mips.c (md_apply_fix)

13
gas/config/tc-mips.c
View File

@ -17165,9 +17165,9 @@ mips_fix_adjustable (fixS *fixp)
There is a further restriction:
5. We cannot reduce jump relocations (R_MIPS_26, R_MIPS16_26 or
R_MICROMIPS_26_S1) against MIPS16 or microMIPS symbols on
targets with in-place addends; the relocation field cannot
encode the low bit.
R_MICROMIPS_26_S1) against MIPS16 or microMIPS symbols because
we need to keep the MIPS16 or microMIPS symbol for the purpose
of converting JAL to JALX instructions in the linker.
For simplicity, we deal with (3)-(4) by not reducing _any_ relocation
against a MIPS16 symbol. We deal with (5) by by not reducing any
@ -17182,10 +17182,9 @@ mips_fix_adjustable (fixS *fixp)
that we have for MIPS16 symbols. */
if (fixp->fx_subsy == NULL
&& (ELF_ST_IS_MIPS16 (S_GET_OTHER (fixp->fx_addsy))
|| *symbol_get_tc (fixp->fx_addsy)
|| (HAVE_IN_PLACE_ADDENDS
&& ELF_ST_IS_MICROMIPS (S_GET_OTHER (fixp->fx_addsy))
&& jmp_reloc_p (fixp->fx_r_type))))
|| (ELF_ST_IS_MICROMIPS (S_GET_OTHER (fixp->fx_addsy))
&& jmp_reloc_p (fixp->fx_r_type))
|| *symbol_get_tc (fixp->fx_addsy)))
return 0;
return 1;

29
gas/testsuite/gas/mips/jalx-local-n32.d
View File

@ -0,0 +1,29 @@
#objdump: -dr --prefix-addresses --show-raw-insn
#name: MIPS JALX local symbol relocation (n32)
#as: -n32 -march=from-abi
#source: jalx-local.s
.*: +file format .*mips.*
Disassembly of section \.text:
\.\.\.
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f400 0000 jal 00000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MICROMIPS_26_S1 \.text\+0x1020
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f000 0000 jalx 00000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MICROMIPS_26_S1 \.text\+0x1020
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 001f 0f3c jr ra
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 0000 0000 nop
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 0c000000 jal 00000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MIPS_26 foo
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 74000000 jalx 00000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MIPS_26 foo
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 03e00009 jalr zero,ra
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
\.\.\.

37
gas/testsuite/gas/mips/jalx-local-n64.d
View File

@ -0,0 +1,37 @@
#objdump: -dr --prefix-addresses --show-raw-insn
#name: MIPS JALX local symbol relocation (n64)
#as: -64 -march=from-abi
#source: jalx-local.s
.*: +file format .*mips.*
Disassembly of section \.text:
\.\.\.
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f400 0000 jal 0000000000000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MICROMIPS_26_S1 \.text\+0x1020
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*\+0x1020
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*\+0x1020
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f000 0000 jalx 0000000000000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MICROMIPS_26_S1 \.text\+0x1020
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*\+0x1020
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*\+0x1020
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 001f 0f3c jr ra
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 0000 0000 nop
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 0c000000 jal 0000000000000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MIPS_26 foo
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 74000000 jalx 0000000000000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MIPS_26 foo
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*
[ ]*[0-9a-f]+: R_MIPS_NONE \*ABS\*
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 03e00009 jalr zero,ra
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
\.\.\.

28
gas/testsuite/gas/mips/jalx-local.d
View File

@ -0,0 +1,28 @@
#objdump: -dr --prefix-addresses --show-raw-insn
#name: MIPS JALX local symbol relocation
#as: -32
.*: +file format .*mips.*
Disassembly of section \.text:
\.\.\.
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f400 0810 jal 00001020 <bar>
[ ]*[0-9a-f]+: R_MICROMIPS_26_S1 \.text
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f000 0408 jalx 00001020 <bar>
[ ]*[0-9a-f]+: R_MICROMIPS_26_S1 \.text
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 001f 0f3c jr ra
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 0000 0000 nop
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 0c000000 jal 00000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MIPS_26 foo
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 74000000 jalx 00000000 <foo-0x1000>
[ ]*[0-9a-f]+: R_MIPS_26 foo
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 03e00009 jalr zero,ra
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
\.\.\.

33
gas/testsuite/gas/mips/jalx-local.s
View File

@ -0,0 +1,33 @@
.text
.set noreorder
.space 0x1000
.align 4
.set micromips
.ent foo
foo:
nor $0, $0
jal bar
nor $0, $0
jalx bar
nor $0, $0
jalr $0, $ra
nor $0, $0
.end foo
.align 4
.set nomicromips
.ent bar
bar:
nor $0, $0
jal foo
nor $0, $0
jalx foo
nor $0, $0
jalr $0, $ra
nor $0, $0
.end bar
# Force some (non-delay-slot) zero bytes, to make 'objdump' print ...
.align 4, 0
.space 16

3
gas/testsuite/gas/mips/mips.exp
View File

@ -796,11 +796,14 @@ if { [istarget mips*-*-vxworks*] } {
run_dump_test "mips-jalx-2"
run_dump_test "jalx-imm"
run_dump_test "jalx-addend"
run_dump_test "jalx-local"
if $has_newabi {
run_dump_test "jalx-imm-n32"
run_dump_test "jalx-addend-n32"
run_dump_test "jalx-local-n32"
run_dump_test "jalx-imm-n64"
run_dump_test "jalx-addend-n64"
run_dump_test "jalx-local-n64"
}
# Check MIPS16 HI16/LO16 relocations
run_dump_test "mips16-hilo"

7
ld/ChangeLog
View File

@ -1,3 +1,10 @@
2016-05-24 Maciej W. Rozycki <macro@imgtec.com>
* testsuite/ld-mips-elf/jalx-local.d: New test.
* testsuite/ld-mips-elf/jalx-local-n32.d: New test.
* testsuite/ld-mips-elf/jalx-local-n64.d: New test.
* testsuite/ld-mips-elf/mips-elf.exp: Run the new tests.
2016-05-23 Kuba Sejdak <jakub.sejdak@phoesys.com>
* Makefile.am: Add earmelf_phoenix.c.

6
ld/testsuite/ld-mips-elf/jalx-local-n32.d
View File

@ -0,0 +1,6 @@
#name: MIPS JALX to local symbol (n32)
#source: ../../../gas/testsuite/gas/mips/jalx-local.s
#as: -EB -n32 -march=from-abi
#ld: -EB -Ttext 0x1c000000 -e 0x1c000000
#objdump: -dr --prefix-addresses --show-raw-insn
#dump: jalx-local.d

6
ld/testsuite/ld-mips-elf/jalx-local-n64.d
View File

@ -0,0 +1,6 @@
#name: MIPS JALX to local symbol (n64)
#source: ../../../gas/testsuite/gas/mips/jalx-local.s
#as: -EB -64 -march=from-abi
#ld: -EB -Ttext 0x1c000000 -e 0x1c000000
#objdump: -dr --prefix-addresses --show-raw-insn
#dump: jalx-local.d

26
ld/testsuite/ld-mips-elf/jalx-local.d
View File

@ -0,0 +1,26 @@
#name: MIPS JALX to local symbol
#source: ../../../gas/testsuite/gas/mips/jalx-local.s
#as: -EB -32
#ld: -EB -Ttext 0x1c000000 -e 0x1c000000
#objdump: -dr --prefix-addresses --show-raw-insn
.*: +file format .*mips.*
Disassembly of section \.text:
\.\.\.
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f300 0408 jalx 0*1c001020 <.*>
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> f300 0408 jalx 0*1c001020 <.*>
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 001f 0f3c jr ra
[0-9a-f]+ <[^>]*> 0000 02d0 not zero,zero
[0-9a-f]+ <[^>]*> 0000 0000 nop
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 77000400 jalx 0*1c001000 <.*>
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 77000400 jalx 0*1c001000 <.*>
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
[0-9a-f]+ <[^>]*> 03e00009 jalr zero,ra
[0-9a-f]+ <[^>]*> 00000027 nor zero,zero,zero
\.\.\.

3
ld/testsuite/ld-mips-elf/mips-elf.exp
View File

@ -169,9 +169,12 @@ if { $linux_gnu } {
}
run_dump_test "jalx-addend" [list [list ld $abi_ldflags(o32)]]
run_dump_test "jalx-local" [list [list ld $abi_ldflags(o32)]]
if $has_newabi {
run_dump_test "jalx-addend-n32" [list [list ld $abi_ldflags(n32)]]
run_dump_test "jalx-local-n32" [list [list ld $abi_ldflags(n32)]]
run_dump_test "jalx-addend-n64" [list [list ld $abi_ldflags(n64)]]
run_dump_test "jalx-local-n64" [list [list ld $abi_ldflags(n64)]]
}
# Test multi-got link. We only do this on GNU/Linux because it requires

Write Preview
Loading…
Cancel
Save