[Bug ld/12942] Plugin not handling correctly resolution of COMDATs.

[cvs-commit at gcc dot gnu.org]

http://sourceware.org/bugzilla/show_bug.cgi?id=12942

--- Comment #4 from cvs-commit at gcc dot gnu.org <cvs-commit at gcc dot 
gnu.org> 2011-07-09 06:20:56 UTC ---
CVSROOT:    /cvs/src
Module name:    src
Changes by:    address@hidden    2011-07-09 06:20:52

Modified files:
    bfd            : ChangeLog elflink.c cofflink.c elf-bfd.h 
                     libbfd-in.h libbfd.h linker.c targets.c 
                     bfd-in2.h 
    include        : ChangeLog bfdlink.h 
    ld             : ChangeLog ldlang.c plugin.c 

Log message:
    PR ld/12942
    bfd/
    * elflink.c (elf_link_add_object_symbols): Use elf_discarded_section
    rather than kept_section to determine whether a symbol is from
    a discarded section.
    * cofflink.c (coff_link_add_symbols): Make symbols from discarded
    sections appear undefined.

    * elf-bfd.h (_bfd_elf_section_already_linked): Replace
    "asection *" with "struct already_linked *".
    * libbfd-in.h (_bfd_nolink_section_already_linked): Likewise.
    (_bfd_generic_section_already_linked): Likewise.
    (bfd_section_already_linked_table_insert): Likewise.
    (struct already_linked): New.
    (struct bfd_section_already_linked): Use it.
    * elflink.c (_bfd_elf_section_already_linked): Replace.
    "asection *" with "struct already_linked *".  Replace the plugin
    dummy with the LTO output.
    * linker.c (_bfd_generic_section_already_linked): Likewise.
    * targets.c (struct already_linked): Add forward declaration.
    (bfd_target): Replace "struct bfd_section *" with
    "struct already_linked *" in _section_already_linked.
    * bfd-in2.h: Regenerate.
    * libbfd.h: Regenerate.

    include/
    * bfdlink.h (bfd_link_info): Add loading_lto_outputs.

    ld/
    * ldlang.c (section_already_linked): Pass "struct already_linked *"
    to bfd_section_already_linked.
    (lang_process): Set link_info.loading_lto_outputs before
    loading LTO outputs.
    * plugin.c: Include "libbfd.h".
    (add_symbols): Call bfd_section_already_linked with comdat_key.

Patches:
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/ChangeLog.diff?cvsroot=src&r1=1.5417&r2=1.5418
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/elflink.c.diff?cvsroot=src&r1=1.412&r2=1.413
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/cofflink.c.diff?cvsroot=src&r1=1.77&r2=1.78
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/elf-bfd.h.diff?cvsroot=src&r1=1.323&r2=1.324
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/libbfd-in.h.diff?cvsroot=src&r1=1.88&r2=1.89
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/libbfd.h.diff?cvsroot=src&r1=1.253&r2=1.254
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/linker.c.diff?cvsroot=src&r1=1.85&r2=1.86
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/targets.c.diff?cvsroot=src&r1=1.202&r2=1.203
http://sourceware.org/cgi-bin/cvsweb.cgi/src/bfd/bfd-in2.h.diff?cvsroot=src&r1=1.540&r2=1.541
http://sourceware.org/cgi-bin/cvsweb.cgi/src/include/ChangeLog.diff?cvsroot=src&r1=1.541&r2=1.542
http://sourceware.org/cgi-bin/cvsweb.cgi/src/include/bfdlink.h.diff?cvsroot=src&r1=1.88&r2=1.89
http://sourceware.org/cgi-bin/cvsweb.cgi/src/ld/ChangeLog.diff?cvsroot=src&r1=1.2350&r2=1.2351
http://sourceware.org/cgi-bin/cvsweb.cgi/src/ld/ldlang.c.diff?cvsroot=src&r1=1.372&r2=1.373
http://sourceware.org/cgi-bin/cvsweb.cgi/src/ld/plugin.c.diff?cvsroot=src&r1=1.35&r2=1.36

-- 
Configure bugmail: http://sourceware.org/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.