[Bug gold/22820] ICF mismerges two very similar functions

[cvs-commit at gcc dot gnu.org]

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

--- Comment #2 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot 
gnu.org> ---
The master branch has been updated by Cary Coutant <address@hidden>:

https://sourceware.org/git/gitweb.cgi?p=binutils-gdb.git;h=651d16203867f8013a0f78a2f2e24df8c02d1377

commit 651d16203867f8013a0f78a2f2e24df8c02d1377
Author: Cary Coutant <address@hidden>
Date:   Tue Apr 24 21:57:37 2018 -0700

    Fix bug with relocation addends and merge sections with --icf.

    During --icf processing, gold was incorrectly processing the relocation
    addend for references to items in a merge section. PC-relative references
    and other forms of reference with a biased base address require a
    non-section local symbol, where the addend is purely the bias.

    gold/
        PR gold/20642
        PR gold/22820
        * gc.h (gc_process_relocs): Flag STT_SECTION symbols in symvec.
        * icf.cc (get_section_contents): For merge sections, ignore the
        addend for relocations against non-section symbols.

-- 
You are receiving this mail because:
You are on the CC list for the bug.