PATCH: PR gas/2117: IA64 assembler is broken
Jan Beulich
JBeulich@novell.com
Mon Jan 9 09:53:00 GMT 2006
>--- gas/symbols.c.equ 2005-12-24 08:16:57.000000000 -0800
>+++ gas/symbols.c 2006-01-06 17:40:57.000000000 -0800
>@@ -1399,7 +1399,9 @@ snapshot_symbol (symbolS **symbolPP, val
> }
> }
>
>- *symbolPP = symbolP;
>+ /* Never change a defined symbol. */
>+ if (symbolP->bsym->section == undefined_section)
>+ *symbolPP = symbolP;
> *valueP = expr.X_add_number;
> *segP = symbolP->bsym->section;
> *fragPP = symbolP->sy_frag;
The minimum, as I view it, is to also include expr_section symbols
here. Beyond that, assuming that you already checked that no existing
test case breaks, I hope this change will not have other ill effects.
Jan
More information about the Binutils
mailing list