This is the mail archive of the
mailing list for the binutils project.
Re: [GOLD] PowerPC64 identical code folding
On Thu, Mar 14, 2013 at 12:25 AM, Alan Modra <firstname.lastname@example.org> wrote:
> On Thu, Mar 14, 2013 at 05:22:15PM +1030, Alan Modra wrote:
>> On Tue, Mar 12, 2013 at 10:50:29AM +1030, Alan Modra wrote:
>> > * gc.h (gc_process_relocs): Look through function descriptors
>> > to determine shndx, symvalue and addend used by ICF. Tidy
>> > variable duplication.
>> On testing this with more than the gold testsuite, I discovered that
>> --icf on PowerPC64 is quite broken. We can't call function_location()
>> for a symbol defined in some object for which do_read_relocs() hasn't
>> been called; The .opd info for that object isn't yet set up.
>> It looks like I'll need to translate shndx, symvalue and addend in
> Like this. OK to apply?
> * gc.h (gc_process_relocs): Don't look through function descriptors.
> * icf.cc (get_section_contents): Do so here instead.
This is OK.