[Bug c++/61654] New: [4.9.1] ICE in release_function_body, at cgraph.c:1699

[Bug c++/61654] New: [4.9.1] ICE in release_function_body, at cgraph.c:1699

[manuel.lauss at googlemail dot com]

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

            Bug ID: 61654
           Summary: [4.9.1] ICE in release_function_body, at cgraph.c:1699
           Product: gcc
           Version: 4.9.0
            Status: UNCONFIRMED
          Severity: normal
          Priority: P3
         Component: c++
          Assignee: unassigned at gcc dot gnu.org
          Reporter: manuel.lauss at googlemail dot com

Created attachment 33032
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33032&action=edit
preprocessed source

current gcc-4_9-branch, ICE with "-O3". -O2 works; gcc-4.8.3 handles "-O3" fine
as well.

(source is scummvm-1.6.0/engines/tony/font.cpp)

i686-pc-linux-gnu-g++ -O3 -c font.i
In file included from ./engines/tony/input.h:32:0,
                 from engines/tony/font.cpp:32:
./common/events.h: In function '<built-in>':
./common/events.h:239:7: internal compiler error: in release_function_body, at
cgraph.c:1699

Thanks!

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[trippels at gcc dot gnu.org]

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

Markus Trippelsdorf <trippels at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|UNCONFIRMED                 |NEW
   Last reconfirmed|                            |2014-06-30
          Component|c++                         |middle-end
                 CC|                            |trippels at gcc dot gnu.org
     Ever confirmed|0                           |1
            Summary|[4.9.1] ICE in              |[4.9/4.10 Regression] ICE
                   |release_function_body, at   |in release_function_body,
                   |cgraph.c:1699               |at cgraph.c:1699
   Target Milestone|---                         |4.9.1
      Known to fail|                            |4.10.0

--- Comment #1 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
markus@x4 tmp % cat font.ii
class A
{
  virtual int m_fn1 (int, int = 0) = 0;
  int m_fn2 (const int &);
  int _nPrior;
};
class B : virtual A
{
  int _lDefault;
  int m_fn1 (int, int);
};
int A::m_fn2 (const int &) { return m_fn1 ('\0'); }

int B::m_fn1 (int, int) { return 0 ?: _lDefault; }


markus@x4 tmp % g++ -m32 -c -O3 font.ii
font.ii: In function ‘<built-in>’:
font.ii:14:50: internal compiler error: in release_function_body, at
cgraph.c:1691
 int B::m_fn1 (int, int) { return 0 ?: _lDefault; }
                                                  ^
0x841577 release_function_body(tree_node*)
        ../../gcc/gcc/cgraph.c:1691
0x841626 cgraph_release_function_body(cgraph_node*)
        ../../gcc/gcc/cgraph.c:1727
0x841aa6 cgraph_remove_node(cgraph_node*)
        ../../gcc/gcc/cgraph.c:1812
0xa1c3eb symtab_remove_unreachable_nodes(bool, _IO_FILE*)
        ../../gcc/gcc/ipa.c:463
0xace957 execute_todo
        ../../gcc/gcc/passes.c:1843
Please submit a full bug report,
with preprocessed source if appropriate.
Please include the complete backtrace with any bug report.
See <http://gcc.gnu.org/bugs.html> for instructions.
>From gcc-bugs-return-455301-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org Mon Jun 30 07:09:46 2014
Return-Path: <gcc-bugs-return-455301-listarch-gcc-bugs=gcc.gnu.org@gcc.gnu.org>
Delivered-To: listarch-gcc-bugs@gcc.gnu.org
Received: (qmail 18912 invoked by alias); 30 Jun 2014 07:09:45 -0000
Mailing-List: contact gcc-bugs-help@gcc.gnu.org; run by ezmlm
Precedence: bulk
List-Id: <gcc-bugs.gcc.gnu.org>
List-Archive: <http://gcc.gnu.org/ml/gcc-bugs/>
List-Post: <mailto:gcc-bugs@gcc.gnu.org>
List-Help: <mailto:gcc-bugs-help@gcc.gnu.org>
Sender: gcc-bugs-owner@gcc.gnu.org
Delivered-To: mailing list gcc-bugs@gcc.gnu.org
Received: (qmail 18843 invoked by uid 55); 30 Jun 2014 07:09:35 -0000
From: "arjen.markus895 at gmail dot com" <gcc-bugzilla@gcc.gnu.org>
To: gcc-bugs@gcc.gnu.org
Subject: [Bug fortran/61628] A program that reads from a file with stream access and uses pack() suddenly stops
Date: Mon, 30 Jun 2014 07:09:00 -0000
X-Bugzilla-Reason: CC
X-Bugzilla-Type: changed
X-Bugzilla-Watch-Reason: None
X-Bugzilla-Product: gcc
X-Bugzilla-Component: fortran
X-Bugzilla-Version: 4.8.1
X-Bugzilla-Keywords:
X-Bugzilla-Severity: normal
X-Bugzilla-Who: arjen.markus895 at gmail dot com
X-Bugzilla-Status: WAITING
X-Bugzilla-Priority: P3
X-Bugzilla-Assigned-To: unassigned at gcc dot gnu.org
X-Bugzilla-Target-Milestone: ---
X-Bugzilla-Flags:
X-Bugzilla-Changed-Fields:
Message-ID: <bug-61628-4-jN3Hclgktu@http.gcc.gnu.org/bugzilla/>
In-Reply-To: <bug-61628-4@http.gcc.gnu.org/bugzilla/>
References: <bug-61628-4@http.gcc.gnu.org/bugzilla/>
Content-Type: text/plain; charset="UTF-8"
Content-Transfer-Encoding: 7bit
X-Bugzilla-URL: http://gcc.gnu.org/bugzilla/
Auto-Submitted: auto-generated
MIME-Version: 1.0
X-SW-Source: 2014-06/txt/msg02383.txt.bz2
Content-length: 1128

https://gcc.gnu.org/bugzilla/show_bug.cgi?ida628

--- Comment #7 from Arjen Markus <arjen.markus895 at gmail dot com> ---
Hi Jerry,

no, it does not - it should have printed all values in the array
(matrix) that are not 1, the first one thousand elements and finally
it should have printed "Done".

The number it does print are the expected ones, but it is about the
numbers and text it does not.

Regards,

Arjen

2014-06-29 23:27 GMT+02:00 jvdelisle at gcc dot gnu.org
<gcc-bugzilla@gcc.gnu.org>:
> https://gcc.gnu.org/bugzilla/show_bug.cgi?ida628
>
> --- Comment #6 from Jerry DeLisle <jvdelisle at gcc dot gnu.org> ---
> I went to gfortran binaries and grabbed the mingw installer and installed it on
> my Win7 side.
>
> Compile the program, Un compressed the data file and ran it.
>
> C:\Users\jerry\prs\pr61628>gfortran pr61628.f90
>
> C:\Users\jerry\prs\pr61628>a
>          681         469      319389
>
> It appears to work fine.  This is with the gfortran 4.10 verrsion as of today.
>
> Maybe try removing gfortran and re-install with todays.
>
> --
> You are receiving this mail because:
> You reported the bug.

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[trippels at gcc dot gnu.org]

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

Markus Trippelsdorf <trippels at gcc dot gnu.org> changed:

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

--- Comment #2 from Markus Trippelsdorf <trippels at gcc dot gnu.org> ---
git blame points to r209892.

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jakub at gcc dot gnu.org

--- Comment #3 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Created attachment 33061
  --> https://gcc.gnu.org/bugzilla/attachment.cgi?id=33061&action=edit
gcc410-pr61654.patch

Untested fix.

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jakub at gcc dot gnu.org]

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

--- Comment #5 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Fri Jul  4 08:38:45 2014
New Revision: 212293

URL: https://gcc.gnu.org/viewcvs?rev=212293&root=gcc&view=rev
Log:
    PR middle-end/61654
    * cgraphunit.c (expand_thunk): Call free_dominance_info.

    * g++.dg/opt/pr61654.C: New test.

Added:
    trunk/gcc/testsuite/g++.dg/opt/pr61654.C
Modified:
    trunk/gcc/ChangeLog
    trunk/gcc/cgraphunit.c
    trunk/gcc/testsuite/ChangeLog

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jakub at gcc dot gnu.org]

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

--- Comment #6 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Author: jakub
Date: Fri Jul  4 08:39:31 2014
New Revision: 212294

URL: https://gcc.gnu.org/viewcvs?rev=212294&root=gcc&view=rev
Log:
    PR middle-end/61654
    * cgraphunit.c (expand_thunk): Call free_dominance_info.

    * g++.dg/opt/pr61654.C: New test.

Added:
    branches/gcc-4_9-branch/gcc/testsuite/g++.dg/opt/pr61654.C
Modified:
    branches/gcc-4_9-branch/gcc/ChangeLog
    branches/gcc-4_9-branch/gcc/cgraphunit.c
    branches/gcc-4_9-branch/gcc/testsuite/ChangeLog

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|ASSIGNED                    |NEW
                 CC|                            |jamborm at gcc dot gnu.org
           Assignee|jakub at gcc dot gnu.org           |unassigned at gcc dot gnu.org

--- Comment #7 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Reduced testcase for the other ICE, ICEs only with -O3 -m32 and not with -O3
-m64 on x86_64-linux:
struct A
{
  virtual int a (int, int = 0);
  void b ();
  void c ();
  int d;
};

struct B : virtual A
{
  int a (int, int);
  int e;
};

int f;

void
A::b ()
{
  a (0);
}

void
A::c ()
{
  a (f);
}

int
B::a (int, int)
{
  return e;
}

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[rguenth at gcc dot gnu.org]

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

--- Comment #8 from Richard Biener <rguenth at gcc dot gnu.org> ---
The reduced testcase doesn't ICE on the branch, the full preprocessed source
does.

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[rguenth at gcc dot gnu.org]

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

Richard Biener <rguenth at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
           Priority|P3                          |P2

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jakub at gcc dot gnu.org]

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

--- Comment #9 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
Richard mentioned that #c7 doesn't ICE on the 4.9 branch, only trunk.  This one
ICEs on the 4.9 branch too (the only change is " = 0", i.e. making A::a pure
virtual), again -O3 -m32 is needed:

struct A
{
  virtual int a (int, int = 0) = 0;
  void b ();
  void c ();
  int d;
};

struct B : virtual A
{
  int a (int, int);
  int e;
};

int f;

void
A::b ()
{
  a (0);
}

void
A::c ()
{
  a (f);
}

int
B::a (int, int)
{
  return e;
}

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jakub at gcc dot gnu.org]

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

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

           What    |Removed                     |Added
----------------------------------------------------------------------------
   Target Milestone|4.9.1                       |4.9.2

--- Comment #10 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
GCC 4.9.1 has been released.

[Bug middle-end/61654] [4.9/4.10 Regression] ICE in release_function_body, at cgraph.c:1699

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |ASSIGNED
           Assignee|unassigned at gcc dot gnu.org      |jamborm at gcc dot gnu.org

--- Comment #11 from Martin Jambor <jamborm at gcc dot gnu.org> ---
ipa_propagate_frequency is seeing an artificial_thunk.  Mine.

[Bug middle-end/61654] [4.9/5 Regression] ICE in release_function_body, at cgraph.c:1699

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
      Known to fail|4.10.0                      |5.0

--- Comment #12 from Martin Jambor <jamborm at gcc dot gnu.org> ---
I have proposed a fix on the mailing list:

https://gcc.gnu.org/ml/gcc-patches/2014-09/msg00207.html

[Bug middle-end/61654] [4.9/5 Regression] ICE in release_function_body, at cgraph.c:1699

[jamborm at gcc dot gnu.org]

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

--- Comment #13 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Author: jamborm
Date: Wed Sep 10 11:34:09 2014
New Revision: 215122

URL: https://gcc.gnu.org/viewcvs?rev=215122&root=gcc&view=rev
Log:
2014-09-10  Martin Jambor  <mjambor@suse.cz>

    PR ipa/61654
    * cgraphclones.c (duplicate_thunk_for_node): Copy arguments of the
    new decl properly.  Analyze the new thunk if it is expanded.

gcc/testsuite/
    * g++.dg/ipa/pr61654.C: New test.


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

[Bug middle-end/61654] [4.9/5 Regression] ICE in release_function_body, at cgraph.c:1699

[jamborm at gcc dot gnu.org]

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

Martin Jambor <jamborm at gcc dot gnu.org> changed:

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

--- Comment #15 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Fixed.

[Bug middle-end/61654] [4.9/5 Regression] ICE in release_function_body, at cgraph.c:1699

[jamborm at gcc dot gnu.org]

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

--- Comment #14 from Martin Jambor <jamborm at gcc dot gnu.org> ---
Author: jamborm
Date: Fri Sep 12 16:52:24 2014
New Revision: 215228

URL: https://gcc.gnu.org/viewcvs?rev=215228&root=gcc&view=rev
Log:
2014-09-12  Martin Jambor  <mjambor@suse.cz>

    PR ipa/61654
    * cgraph.h (cgraph_analyze_function): Declare.
    * cgraphunit.c: (analyze_function): Remove forward declaration,
    rename to cgraph_analyze_function, made external.
    * cgraphclones.c (duplicate_thunk_for_node): Copy arguments of the
        new decl properly.  Analyze the new thunk if it is expanded.

gcc/testsuite/
        * g++.dg/ipa/pr61654.C: New test.


Added:
    branches/gcc-4_9-branch/gcc/testsuite/g++.dg/ipa/pr61654.C
Modified:
    branches/gcc-4_9-branch/gcc/ChangeLog
    branches/gcc-4_9-branch/gcc/cgraph.h
    branches/gcc-4_9-branch/gcc/cgraphclones.c
    branches/gcc-4_9-branch/gcc/cgraphunit.c
    branches/gcc-4_9-branch/gcc/testsuite/ChangeLog