FreeChainXenon/binutils-gdb
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

* readelf.c (debug_apply_rela_addends): Clarify FIXME.

Browse source
This commit is contained in:
Alan Modra 2007-10-31 07:40:11 +00:00
parent 8f317e3169
commit ebc056ca1b
2 changed files with 11 additions and 8 deletions
Download patch file Download diff file Expand all files Collapse all files

4
binutils/ChangeLog
View file

@ -1,3 +1,7 @@
2007-10-31 Alan Modra <amodra@bigpond.net.au>
* readelf.c (debug_apply_rela_addends): Clarify FIXME.
2007-10-29 Nick Clifton <nickc@redhat.com> 2007-10-29 Nick Clifton <nickc@redhat.com>
* readelf.c (is_32bit_abs_reloc): Add knowledge of reloc used by * readelf.c (is_32bit_abs_reloc): Add knowledge of reloc used by

13
binutils/readelf.c
View file

@ -8242,13 +8242,12 @@ debug_apply_rela_addends (void *file,
continue; continue;
} }
if (is_32bit_pcrel_reloc (reloc_type)) /* FIXME. We apply pcrel relocs as if they were absolute,
/* FIXME: Not sure how to apply a pc-rel reloc yet. ie. without subtracting the pc. This is to suit
I think that it ought to be: display_debug_frames which does not add the pc even
(rp->r_addend + sym->st_value) - rp->r_offset though it ought to for DW_EH_PE_pcrel FDEs. Removing
but this breaks GAS CFI tests... */ the hack in display_debug_frames will require that we
byte_put (loc, (rp->r_addend + sym->st_value) /*- rp->r_offset*/, reloc_size); apply rel relocs too. */
else
byte_put (loc, rp->r_addend + sym->st_value, reloc_size); byte_put (loc, rp->r_addend + sym->st_value, reloc_size);
} }