https://sourceware.org/bugzilla/show_bug.cgi?id=20427
Nick Clifton <nickc at redhat dot com> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |nickc at redhat dot com
--- Comment #3 from Nick Clifton <nickc at redhat dot com> ---
Hi Stefan,
> proposed patch -- tc-sparc.c.patch
I have a couple of questions about this patch:
1. There appear to be other places in cons_fix_new_sparc() where
64-bit relocs can be generated (ie BFD_RELOC_64_PCREL,
BFD_RELOC_SPARC_PLT64 and BFD_RELOC_SPARC_TLS_DTPOFF64) but the
patch does not affect them. Why not ?
2. The relocs that you are changing are intended to install 64-bit
values into the binary. By changing them to 32-bit versions it
seems like you are loosing the upper bits. Have you run any tests
to see if 64-bit values are still handled correctly after applying
the patch ? [It would be useful to know what the Solaris assembler
does in these circumstances. Does it, for example, produce two
32-bit relocs, one for the lower 32-bits and one for the upper
32-bits ?]
Cheers
Nick
--
You are receiving this mail because:
You are on the CC list for the bug.
_______________________________________________
bug-binutils mailing list
bug-binutils@gnu.org
https://lists.gnu.org/mailman/listinfo/bug-binutils
