[Bug ipa/60457] New: [4.9 Regression] ICE in cgraph_get_node

[Bug ipa/60457] New: [4.9 Regression] ICE in cgraph_get_node

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

            Bug ID: 60457
           Summary: [4.9 Regression] ICE in cgraph_get_node
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: ipa
          Assignee: unassigned at gcc dot gnu.org
          Reporter: jakub at gcc dot gnu.org

template <class T>
struct A
{
};
struct B : A <B>
{
  B ();
};
B::B ()
{
  const int c[] = { 1, 1 };
}

ICEs at -O0 starting with r201408.

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-03-07
   Target Milestone|---                         |4.9.0
     Ever confirmed|0                           |1

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[mpolacek at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

Marek Polacek <mpolacek at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P1
                 CC|                            |mpolacek at gcc dot gnu.org

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

--- Comment #1 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 32302
  --> http://gcc.gnu.org/bugzilla/attachment.cgi?id=32302&action=edit
gcc49-pr60457.patch

Perhaps something like this can fix this?  From the patch description it seems
you were after functions only.

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |hubicka at gcc dot gnu.org

--- Comment #2 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Honza?

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[rguenth at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

--- Comment #3 from Richard Biener <rguenth at gcc dot gnu.org> ---
Looks obvious to me.

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

--- Comment #4 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Mon Mar 10 14:55:20 2014
New Revision: 208454

URL: http://gcc.gnu.org/viewcvs?rev=208454&root=gcc&view=rev
Log:
    PR ipa/60457
    * ipa.c (symtab_remove_unreachable_nodes): Don't call
    cgraph_get_create_node on VAR_DECLs.

    * g++.dg/ipa/pr60457.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/ipa/pr60457.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/ipa.c
    trunk/gcc/testsuite/ChangeLog

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[jakub at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

Jakub Jelinek <jakub at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Fixed.

[Bug ipa/60457] [4.9 Regression] ICE in cgraph_get_node

[hubicka at gcc dot gnu.org]

http://gcc.gnu.org/bugzilla/show_bug.cgi?id=60457

--- Comment #6 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
Yes, it is obvious.  We will however need to get better origin representation
(supporting early inlined functions and probably also the variable origins as
they apparently can exist) later on if we ever want save debug info with LTO.

Thanks,
Honza