[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Martin Liška  changed:

   What|Removed |Added

 Status|ASSIGNED|RESOLVED
 Resolution|--- |FIXED

--- Comment #15 from Martin Liška  ---
Fixed.

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #14 from Martin Liška  ---
Author: marxin
Date: Fri Jan 20 08:44:35 2017
New Revision: 244687

URL: https://gcc.gnu.org/viewcvs?rev=244687=gcc=rev
Log:
Fix IPA CP where it forgot to add a reference in cgraph (PR ipa/71190).

2017-01-20  Martin Liska  

PR ipa/71190
* cgraph.h (maybe_create_reference): Remove argument and
update comment.
* cgraphclones.c (cgraph_node::create_virtual_clone): Remove one
argument.
* ipa-cp.c (create_specialized_node): Likewise.
* symtab.c (symtab_node::maybe_create_reference): Handle
VAR_DECLs and ADDR_EXPRs and select ipa_ref_use type.

Modified:
trunk/gcc/ChangeLog
trunk/gcc/cgraph.h
trunk/gcc/cgraphclones.c
trunk/gcc/ipa-cp.c
trunk/gcc/symtab.c

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Martin Liška  changed:

   What|Removed |Added

   Assignee|hubicka at gcc dot gnu.org |marxin at gcc dot 
gnu.org

--- Comment #13 from Martin Liška  ---
Ok, it's caused by IPA CP, will be fixed by:
https://gcc.gnu.org/ml/gcc-patches/2017-01/msg00609.html

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #12 from Markus Trippelsdorf  ---
(In reply to Martin Liška from comment #11)
> Do you have still Markus the pre-processed source files available on the
> machine? Can you give me access?

No, I removed them some time ago. 
But, as I wrote, to reproduce just build Firefox on gcc112 with
""-flto=60 --param lto-partitions=60".

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[marxin at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #11 from Martin Liška  ---
Do you have still Markus the pre-processed source files available on the
machine? Can you give me access?

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[doko at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Matthias Klose  changed:

   What|Removed |Added

 CC||doko at gcc dot gnu.org

--- Comment #10 from Matthias Klose  ---
saw this issue here as well on ARM32 hard float and powerpc, r243559:

https://launchpad.net/ubuntu/+archive/test-rebuild-20161202-gcc7/+build/11456861
https://launchpad.net/ubuntu/+archive/test-rebuild-20161202-gcc7/+build/11456863

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[hubicka at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Jan Hubicka  changed:

   What|Removed |Added

 Status|NEW |ASSIGNED
   Assignee|unassigned at gcc dot gnu.org  |hubicka at gcc dot 
gnu.org

--- Comment #9 from Jan Hubicka  ---
Let me see if I can reproduce this.

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #8 from Markus Trippelsdorf  ---
(In reply to Jakub Jelinek from comment #7)
> Can this still be reproduced with latest trunk?

Yes.

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[jakub at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Jakub Jelinek  changed:

   What|Removed |Added

 CC||jakub at gcc dot gnu.org

--- Comment #7 from Jakub Jelinek  ---
Can this still be reproduced with latest trunk?

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #6 from Markus Trippelsdorf  ---
OK. I've given up on reducing.

Honza, you can find the unreduced testcase on gcc112 in
/home/trippels/lto_ice_testcase :

trippels@gcc2-power8 lto_ice_testcase % ~/gcc_test/usr/local/bin/c++ -w
-flto=60 -flto-partition=max -ffunction-sections -fdata-sections
-fno-exceptions -fno-strict-aliasing -fno-rtti -fno-exceptions -fno-math-errno
-std=gnu++0x -pthread -pipe -O3 -fomit-frame-pointer -fPIC -shared *.ii
lto1: internal compiler error: in assemble_variable_contents, at varasm.c:2065

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #5 from Markus Trippelsdorf  ---
I'm down to 16 *.ii files. It will probably take a few days to creduce them
all.

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[trippels at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

--- Comment #4 from Markus Trippelsdorf  ---
In my case it only happens with "-flto=60 --param lto-partitions=60",
-flto=60 alone works fine.

So perhaps flto-partition=max will help reducing. (Will give it another try
later
today myself.)

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[pinskia at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Andrew Pinski  changed:

   What|Removed |Added

   Keywords||ice-on-valid-code, lto

--- Comment #3 from Andrew Pinski  ---
Any news on reducing it?

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[ktkachov at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

ktkachov at gcc dot gnu.org changed:

   What|Removed |Added

 Status|UNCONFIRMED |NEW
   Last reconfirmed||2016-06-06
 CC||ktkachov at gcc dot gnu.org
 Ever confirmed|0   |1

--- Comment #2 from ktkachov at gcc dot gnu.org ---
I hit this ICE on aarch64.
The testcase I have is quite large, waiting for creduce to chomp on it.

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[jamborm at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Martin Jambor  changed:

   What|Removed |Added

 CC||hubicka at gcc dot gnu.org

--- Comment #1 from Martin Jambor  ---
When we briefly talked about this with Honza on Friday, he had quite a
few relevant and important insights, followed up by questions on IRC
which however seem to be lost.  Honza, can you please ask them here?

[Bug ipa/71190] [7 Regression] ICE in assemble_variable_contents, at varasm.c:2054

[rguenth at gcc dot gnu.org]

https://gcc.gnu.org/bugzilla/show_bug.cgi?id=71190

Richard Biener  changed:

   What|Removed |Added

   Target Milestone|--- |7.0