From 94d4433aa6a50ca4258164e33f6f46fdf4637346 Mon Sep 17 00:00:00 2001 From: Alan Modra Date: Tue, 5 Feb 2013 23:02:54 +0000 Subject: [PATCH] PR gas/14255 * config/tc-avr.h (TC_VALIDATE_FIX): Mark symbol used by reloc. Don't skip fixups with fx_subsy non-NULL. * config/tc-avr.c (tc_gen_reloc): Don't specially handle fixups with fx_subsy non-NULL. --- gas/ChangeLog | 8 ++++++++ gas/config/tc-avr.c | 24 ++---------------------- gas/config/tc-avr.h | 12 +++++++----- 3 files changed, 17 insertions(+), 27 deletions(-) diff --git a/gas/ChangeLog b/gas/ChangeLog index 45014a6d2a9..d064176ebd6 100644 --- a/gas/ChangeLog +++ b/gas/ChangeLog @@ -1,3 +1,11 @@ +2013-02-06 Alan Modra + + PR gas/14255 + * config/tc-avr.h (TC_VALIDATE_FIX): Mark symbol used by reloc. + Don't skip fixups with fx_subsy non-NULL. + * config/tc-avr.c (tc_gen_reloc): Don't specially handle fixups + with fx_subsy non-NULL. + 2013-02-04 H.J. Lu * doc/c-metag.texi: Add "@c man" markers. diff --git a/gas/config/tc-avr.c b/gas/config/tc-avr.c index c6931203c31..0bc972e4b02 100644 --- a/gas/config/tc-avr.c +++ b/gas/config/tc-avr.c @@ -1386,29 +1386,9 @@ tc_gen_reloc (asection *seg ATTRIBUTE_UNUSED, { arelent *reloc; - if (fixp->fx_addsy && fixp->fx_subsy) + if (fixp->fx_subsy != NULL) { - long value = 0; - - if ((S_GET_SEGMENT (fixp->fx_addsy) != S_GET_SEGMENT (fixp->fx_subsy)) - || S_GET_SEGMENT (fixp->fx_addsy) == undefined_section) - { - as_bad_where (fixp->fx_file, fixp->fx_line, - "Difference of symbols in different sections is not supported"); - return NULL; - } - - /* We are dealing with two symbols defined in the same section. - Let us fix-up them here. */ - value += S_GET_VALUE (fixp->fx_addsy); - value -= S_GET_VALUE (fixp->fx_subsy); - - /* When fx_addsy and fx_subsy both are zero, md_apply_fix - only takes it's second operands for the fixup value. */ - fixp->fx_addsy = NULL; - fixp->fx_subsy = NULL; - md_apply_fix (fixp, (valueT *) &value, NULL); - + as_bad_where (fixp->fx_file, fixp->fx_line, _("expression too complex")); return NULL; } diff --git a/gas/config/tc-avr.h b/gas/config/tc-avr.h index e82bd01c192..ee36e65b5aa 100644 --- a/gas/config/tc-avr.h +++ b/gas/config/tc-avr.h @@ -125,9 +125,9 @@ extern long md_pcrel_from_section (struct fix *, segT); /* We don't want gas to fixup the following program memory related relocations. We will need them in case that we want to do linker relaxation. We could in principle keep these fixups in gas when not relaxing. - However, there is no serious performance penilty when making the linker + However, there is no serious performance penalty when making the linker make the fixup work. Check also that fx_addsy is not NULL, in order to make - sure that the fixup refers to some sort of lable. */ + sure that the fixup refers to some sort of label. */ #define TC_VALIDATE_FIX(FIXP,SEG,SKIP) \ if ( (FIXP->fx_r_type == BFD_RELOC_AVR_7_PCREL \ || FIXP->fx_r_type == BFD_RELOC_AVR_13_PCREL \ @@ -143,9 +143,11 @@ extern long md_pcrel_from_section (struct fix *, segT); || FIXP->fx_r_type == BFD_RELOC_AVR_8_HI \ || FIXP->fx_r_type == BFD_RELOC_AVR_8_HLO \ || FIXP->fx_r_type == BFD_RELOC_AVR_16_PM) \ - && (FIXP->fx_addsy)) \ - { \ - goto SKIP; \ + && FIXP->fx_addsy != NULL \ + && FIXP->fx_subsy == NULL) \ + { \ + symbol_mark_used_in_reloc (FIXP->fx_addsy); \ + goto SKIP; \ } /* This target is buggy, and sets fix size too large. */ -- 2.30.2