Hi,
We have decided to apply the forwarded patch to the embedded-7-branch to fix an
ICE when doing partial LTO with weak symbols.
ChangeLog entry is as follows:
2017-11-28 Thomas Preud'homme
Backport from mainline
2017-06-15 Jan Hubicka
Thomas Preud'
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
Backport from mainline
2017-06-15 Jan Hubicka
> 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
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 "" } */
directive.
TIA
Dominique
Hello,
I apologize for absurdly long time to check the patch. I misread the testcase
originally and got confused. Now it makes sense to me.
It does not make sense to keep the alias in symtab when we know that it
was prevailed by external definition. So lto-symtab should turn it into
normal exter
Ping?
Best regards,
Thomas
On 06/06/17 11:12, Thomas Preudhomme wrote:
On 09/05/17 23:36, Jan Hubicka wrote:
Ping?
Sorry for late reply
My turn to apologize now.
Hi,
This patch fixes an assert failure when linking one LTOed object file
having a weak alias with a regular object file cont
