--- Comment #16 from rguenth at gcc dot gnu dot org 2010-04-15 13:44
---
Fixed.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNE
--- Comment #15 from rguenth at gcc dot gnu dot org 2010-04-15 13:44
---
Subject: Bug 43611
Author: rguenth
Date: Thu Apr 15 13:44:31 2010
New Revision: 158376
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=158376
Log:
2010-04-15 Richard Guenther
PR c++/43611
--- Comment #13 from rguenth at gcc dot gnu dot org 2010-04-06 11:21
---
GCC 4.5.0 is being released. Deferring to 4.5.1.
--
rguenth at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #12 from jason at redhat dot com 2010-04-03 21:12 ---
Subject: Re: [4.5 Regression] ICE: SIGSEGV with
-fipa-cp-clone -fkeep-inline-functions
On 04/03/2010 05:10 PM, rguenther at suse dot de wrote:
> But the initial patch also regresses g++.dg/opt/inline8.C, as for
> the in
--- Comment #11 from rguenther at suse dot de 2010-04-03 21:10 ---
Subject: Re: [4.5 Regression] ICE: SIGSEGV with
-fipa-cp-clone -fkeep-inline-functions
On Sat, 3 Apr 2010, jason at gcc dot gnu dot org wrote:
> --- Comment #10 from jason at gcc dot gnu dot org 2010-04-03 21:02
--- Comment #10 from jason at gcc dot gnu dot org 2010-04-03 21:02 ---
If we're going to mess with that code at all, I'd prefer your initial patch.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43611
--- Comment #9 from hubicka at ucw dot cz 2010-04-03 20:52 ---
Subject: Re: [4.5 Regression] ICE: SIGSEGV
with -fipa-cp-clone -fkeep-inline-functions
> The patch probably only papers over the problem. Honza, can you have a look
> here?
Hi,
I am away for Easter till Monday, but
--- Comment #8 from rguenth at gcc dot gnu dot org 2010-04-03 20:12 ---
Both work with
Index: gcc/cp/semantics.c
===
--- gcc/cp/semantics.c (revision 157953)
+++ gcc/cp/semantics.c (working copy)
@@ -3449,7 +3449,9 @@ exp
--- Comment #7 from rguenth at gcc dot gnu dot org 2010-04-03 18:40 ---
But g++.dg/opt/inline8.C then fails. Why's foo DECL_EXTERNAL?
>
QI
size
unit size
align 8 symtab 0 alias set -1 canonical type 0xb77428a0
arg-types >>
nothrow public s
--- Comment #6 from rguenth at gcc dot gnu dot org 2010-04-03 17:11 ---
Looking at the testcase the issue is obvious. We have marked the
instantiations as extern, and indeed the cgraph code dealing with
-fkeep-inline-functions honors this and does not preserve extern
inline bodies while
--- Comment #5 from rguenth at gcc dot gnu dot org 2010-04-02 17:43 ---
The patch probably only papers over the problem. Honza, can you have a look
here?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43611
--- Comment #4 from jason at redhat dot com 2010-04-01 15:55 ---
Subject: Re: [4.5 Regression] ICE: SIGSEGV with
-fipa-cp-clone -fkeep-inline-functions
On 04/01/2010 09:39 AM, rguenth at gcc dot gnu dot org wrote:
> The issue seems to be the C++ frontend marking inline functions neede
--- Comment #3 from rguenth at gcc dot gnu dot org 2010-04-01 13:39 ---
The issue seems to be the C++ frontend marking inline functions needed
at cp/semantics.c:3452 while the middle-end should already take care
of that in cgraph_decide_is_function_needed.
Somehow that confuses us and w
--- Comment #2 from zsojka at seznam dot cz 2010-04-01 13:29 ---
Created an attachment (id=20274)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20274&action=view)
reduced testcase
Command line:
gcc -O3 -fkeep-inline-functions pr43611.C
or
gcc -O2 -fipa-cp-clone -fkeep-inline-funct
--- Comment #1 from zsojka at seznam dot cz 2010-04-01 06:41 ---
Created an attachment (id=20270)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=20270&action=view)
auto-reduced testcase
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=43611
15 matches
Mail list logo
