https://sourceware.org/bugzilla/show_bug.cgi?id=16177

James Clarke <jrtc27 at jrtc27 dot com> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |jrtc27 at jrtc27 dot com

--- Comment #9 from James Clarke <jrtc27 at jrtc27 dot com> ---
Created attachment 11124
  --> https://sourceware.org/bugzilla/attachment.cgi?id=11124&action=edit
Updated/fixed patch

The original patch on this bug report looks at whether the symbol in question
is in a read-only section, but that should have no effect on whether this
symbol needs to be copied into the executable. Instead, it should be checking
whether there are any relocations in read-only sections referring to the
symbol, as that determines whether we are able to leave in dynamic relocations.
Ben's rebased version also has the issue of messing with the SEC_READONLY check
currently present; that should stay, as SEC_READONLY determines where the
copied symbol should go, but only matters if we are doing a copy in the first
place. This patch hasn't even been compile tested, but I'm 90% confident it
works!

-- 
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

Reply via email to