This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

[arm-embedded] [PATCH, GCC/LTO, ping] Fix PR69866: LTO with def for weak alias in regular object file


Hi,

We have decided to apply the referenced fix (r249352) to the ARM/embedded-6-branch along with its initial commit (r249224) to fix an ICE with LTO and aliases.

Fix PR69866

    2017-06-20  Thomas Preud'homme  <thomas.preudhomme@arm.com>

        Backport from mainline
        2017-06-15  Jan Hubicka  <hubicka@ucw.cz>
                    Thomas Preud'homme  <thomas.preudhomme@arm.com>

        gcc/
        PR lto/69866
        * lto-symtab.c (lto_symtab_merge_symbols): Drop useless definitions
        that resolved externally.
        2017-06-15  Thomas Preud'homme  <thomas.preudhomme@arm.com>

        gcc/testsuite/
        PR lto/69866
        * gcc.dg/lto/pr69866_0.c: New test.
        * gcc.dg/lto/pr69866_1.c: Likewise.

        Backport from mainline
        2017-06-18  Jan Hubicka  <hubicka@ucw.cz>

        gcc/testsuite/
        * gcc.dg/lto/pr69866_0.c: This test needs alias.

Best regards,

Thomas
--- Begin Message ---
> The new test fails on darwin with the usual
> 
> FAIL: gcc.dg/lto/pr69866 c_lto_pr69866_0.o-c_lto_pr69866_1.o link, -O0 -flto -flto-partition=none
> 
> IMO it requires a
> 
> /* { dg-require-alias "" } */

Yep,I will add it shortly.

Honza
> 
> directive.
> 
> TIA
> 
> Dominique

--- End Message ---

Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]