This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[arm-embedded] [PATCH, GCC/LTO, ping] Fix PR69866: LTO with def for weak alias in regular object file
- From: Thomas Preudhomme <thomas dot preudhomme at foss dot arm dot com>
- To: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 20 Jun 2017 14:20:55 +0100
- Subject: [arm-embedded] [PATCH, GCC/LTO, ping] Fix PR69866: LTO with def for weak alias in regular object file
- Authentication-results: sourceware.org; auth=none
- References: <20170618195601.GG30427@atrey.karlin.mff.cuni.cz>
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 ---
- From: Jan Hubicka <hubicka at ucw dot cz>
- To: Dominique d'Humi??res <dominiq at lps dot ens dot fr>
- Cc: hubicka Jan <hubicka at ucw dot cz>, gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Sun, 18 Jun 2017 21:56:01 +0200
- Subject: Re: [PATCH, GCC/LTO, ping] Fix PR69866: LTO with def for weak alias in regular object file
- Authentication-results: sourceware.org; auth=none
- Dkim-signature: v=1; a=rsa-sha1; c=relaxed; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; s=default; bh=ncJYauqSv+3NZErFHsP7P0x B7bM=; b=tMnDsjwmHiU25aDAhuoqg4ix96nnvuLzl4Jj5F54+wQ1V1sqdV2Hmd6 akUdGub5wwmK8X/r0JfVBXBeAKDcDmQTZF9iwJm11MmBIGzAayl50VBnDE+vPXGf IVMozihO/blBT4KTujdg5Jc7pdOodSPD0fFNLkLOEyageK05kX6k=
- Domainkey-signature: a=rsa-sha1; c=nofws; d=gcc.gnu.org; h=list-id :list-unsubscribe:list-archive:list-post:list-help:sender:date :from:to:cc:subject:message-id:references:mime-version :content-type:in-reply-to; q=dns; s=default; b=UOtUBAYqjNGP+iW0e ht0i1/t0Fmb81YlDStLLFC3GiqLBvzDUxa6EZeiMr0AaRr2aignnZw3fMKWlIAHm KhJfqkCrX9DPs2xsmAeNgEFRv8BgPMLt7wL5anDg3wPpoFBXo5nyyYYtAWw9ZdCN 4xjycn8VzJACWSO7UufpvzN9Ms=
- References: <D3D5AD20-7A81-4550-87A0-33260BC41450@lps.ens.fr>
> 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 ---