From 7f632ae6db8bfa2dd764048d068364d6a97fe89e Mon Sep 17 00:00:00 2001 Message-Id: <7f632ae6db8bfa2dd764048d068364d6a97fe89e.1406876013.git.olegendo@gcc.gnu.org> From: Oleg Endo Date: Fri, 1 Aug 2014 08:52:41 +0200 Subject: [PATCH] Fix PR 10378. --- gas/ChangeLog | 6 ++++++ gas/config/tc-sh.c | 2 +- ld/testsuite/ChangeLog | 6 ++++++ ld/testsuite/ld-sh/adjsw8.s | 16 ++++++++++++++++ ld/testsuite/ld-sh/sh.exp | 20 ++++++++++++++++++++ 5 files changed, 49 insertions(+), 1 deletion(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 72d54db..b7926f4 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,9 @@ +2014-08-01 Takashi Yoshii + + PR 10378 + * config/tc-sh.c (tc_gen_reloc): Fix initialization of addend in + SWITCH_TABLE case. + 2014-07-29 Matthew Fortune * config/tc-mips.c: Rename INSN_LOAD_COPROC_DELAY to INSN_LOAD_COPROC diff --git a/gas/config/tc-sh.c b/gas/config/tc-sh.c index a0cd212..6e9ae92 100644 --- a/gas/config/tc-sh.c +++ b/gas/config/tc-sh.c @@ -4435,7 +4435,7 @@ tc_gen_reloc (asection *section ATTRIBUTE_UNUSED, fixS *fixp) if (SWITCH_TABLE (fixp)) { *rel->sym_ptr_ptr = symbol_get_bfdsym (fixp->fx_subsy); - rel->addend = 0; + rel->addend = rel->address - S_GET_VALUE(fixp->fx_subsy); if (r_type == BFD_RELOC_16) r_type = BFD_RELOC_SH_SWITCH16; else if (r_type == BFD_RELOC_8) diff --git a/ld/testsuite/ChangeLog b/ld/testsuite/ChangeLog index 02f9dc4..773a638 100644 --- a/ld/testsuite/ChangeLog +++ b/ld/testsuite/ChangeLog @@ -1,3 +1,9 @@ +2014-08-01 Takashi Yoshii + + PR 10378 + * ld-sh/adjsw8.s: New. + * ld-sh/sh.exp: Add switch8 adjustment test. + 2014-07-29 Matthew Fortune * ld-mips-elf/abiflags-strip1-ph.d: New. diff --git a/ld/testsuite/ld-sh/adjsw8.s b/ld/testsuite/ld-sh/adjsw8.s new file mode 100644 index 0000000..6518e26 --- /dev/null +++ b/ld/testsuite/ld-sh/adjsw8.s @@ -0,0 +1,16 @@ +L0: .byte L1-L0 + .byte L2-L0 + .byte L3-L0 + .align 1 +L1: mov.l 1f,r1 + .uses L1 + jmp @r1 + nop +L2: mov.l 1f,r1 + .uses L2 + jmp @r1 + nop +L3: nop + .align 2 +1: .long 2f +2: diff --git a/ld/testsuite/ld-sh/sh.exp b/ld/testsuite/ld-sh/sh.exp index 31181a9..a270400 100644 --- a/ld/testsuite/ld-sh/sh.exp +++ b/ld/testsuite/ld-sh/sh.exp @@ -110,6 +110,26 @@ if ![ld_simple_link $ld tmpdir/sh1.s1 $srec_relax_arg ] { } } +set testadjsw8 "SH switch8 adjustment after relax" +if ![ld_assemble $as "-relax $srcdir/$subdir/adjsw8.s" tmpdir/adjsw8.o] { + unresolved $testadjsw8 +} else { + if ![ld_simple_link $ld tmpdir/adjsw8 "-relax tmpdir/adjsw8.o"] { + fail $testadjsw8 + } else { + send_log "exec $objdump -s tmpdir/adjsw8\n" + verbose "exec $objdump -s tmpdir/adjsw8" + catch "exec $objdump -s tmpdir/adjsw8" exec_output + if [string match "*04080c00*" $exec_output] { + pass $testadjsw8 + } else { + send_log "bad switch table\n" + verbose "bad switch table" + fail $testadjsw8 + } + } +} + set testlink "SH relaxing" set testjsr "SH confirm relaxing" set testrun "SH relaxing execution" -- 1.7.9.5