From b131d1fcfa5978f105a9ac39057475f54bab9202 Mon Sep 17 00:00:00 2001 From: Kaz Kojima Date: Fri, 1 Aug 2014 19:13:39 +0900 Subject: [PATCH] Fix PR10378 which is SH relax bug. --- 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(-) create mode 100644 ld/testsuite/ld-sh/adjsw8.s diff --git a/gas/ChangeLog b/gas/ChangeLog index 72d54dbb7f8..b7926f45aa0 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 a0cd212641e..6e9ae921deb 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 02f9dc4f068..773a6389e86 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 00000000000..6518e26beaf --- /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 31181a945f8..a2704005db8 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"