--- Comment #26 from pinskia at gcc dot gnu dot org 2006-02-12 16:20
---
Fixed by a simplier patch.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
--- Comment #25 from jason at gcc dot gnu dot org 2006-02-12 16:07 ---
Subject: Bug 24996
Author: jason
Date: Sun Feb 12 16:07:04 2006
New Revision: 110890
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110890
Log:
PR c++/24996
* except.c (build_throw): Add a CLE
--- Comment #24 from jason at gcc dot gnu dot org 2006-02-12 16:02 ---
Subject: Bug 24996
Author: jason
Date: Sun Feb 12 16:02:00 2006
New Revision: 110889
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=110889
Log:
PR c++/24996
* except.c (build_throw): Add a CLE
--- Comment #23 from jason at redhat dot com 2006-02-12 07:58 ---
Subject: Re: [4.0/4.1/4.2 Regression] ICE on throw code
I think I have a better patch that I'll check in soon.
Jason
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24996
--- Comment #22 from steven at gcc dot gnu dot org 2006-02-09 22:59 ---
Fascinating how quickly review happens for a patch for a P1 bug report blocking
a release
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24996
--- Comment #21 from rakdver at gcc dot gnu dot org 2006-02-02 17:57
---
I have posted the patch, let's see what the reactions will be.
http://gcc.gnu.org/ml/gcc-patches/2006-02/msg00146.html
--
rakdver at gcc dot gnu dot org changed:
What|Removed
--- Comment #20 from matz at suse dot de 2006-02-02 16:56 ---
I've put the patch to testing.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24996
--- Comment #19 from mark at codesourcery dot com 2006-02-01 08:21 ---
Subject: Re: [4.0/4.1/4.2 Regression] ICE on throw code
rakdver at atrey dot karlin dot mff dot cuni dot cz wrote:
>>Zdenek, have you submitted the patch yet for mainline?
>
> no, I was waiting for reactions on my
--- Comment #18 from rakdver at atrey dot karlin dot mff dot cuni dot cz
2006-02-01 08:18 ---
Subject: Re: [4.0/4.1/4.2 Regression] ICE on throw code
Hello,
> Zdenek, have you submitted the patch yet for mainline?
no, I was waiting for reactions on my questions, so that I am sure
1
--- Comment #17 from mmitchel at gcc dot gnu dot org 2006-02-01 03:27
---
Zdenek, have you submitted the patch yet for mainline?
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24996
--- Comment #16 from rguenth at gcc dot gnu dot org 2006-01-31 16:14
---
Can you quantify "a bit worse" code? Also, did you bootstrap/test the patch?
Btw., now that it's properly analyzed I don't think this should be P1 - in fact
the patch looks too invasive for 4.1.0, but it should p
--- Comment #15 from rakdver at gcc dot gnu dot org 2006-01-26 23:52
---
The patch fixes the problem by making gimplification of cleanups much more
robust, and able to handle nested statements, at the expense of producing a bit
worse code.
--
http://gcc.gnu.org/bugzilla/show_bug.cg
--- Comment #14 from rakdver at gcc dot gnu dot org 2006-01-26 23:51
---
Created an attachment (id=10738)
--> (http://gcc.gnu.org/bugzilla/attachment.cgi?id=10738&action=view)
Possible patch
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24996
--- Comment #13 from rakdver at gcc dot gnu dot org 2006-01-23 08:53
---
One more question; how is the throw code supposed to work?
We allocate D.2069, initialize it, if there is a problem
during it initialization, we deallocate it, then
pass it to __cxa_throw anyway? This does not mak
--- Comment #12 from mmitchel at gcc dot gnu dot org 2006-01-23 05:10
---
I don't fully understand stabilize_expr; that's Jason's invention, IIRC.
However, I think that the problem is that for a COND_EXPR we can't pre-evaluate
both arms of the conditional, because only one of them is s
--- Comment #11 from rakdver at gcc dot gnu dot org 2006-01-22 21:49
---
This ICE works this way: to build_throw, we get expression exp that looks like
TARGET_EXPR
Where t1 and t2 are TARGET_EXPRs with cleanups (*). build_throw generates code
TARGET_EXPR ;
try
{
*(struct logi
--- Comment #10 from steven at gcc dot gnu dot org 2006-01-21 22:46 ---
Zdenek, are you working on a patch for this?
--
steven at gcc dot gnu dot org changed:
What|Removed |Added
-
--- Comment #9 from rakdver at gcc dot gnu dot org 2006-01-02 18:34 ---
As the comments in gimplify_cleanup_point_expr indicate,
it only handles cases when cleanup_exprs are not within nested
constructs (with the exception of conditions).
In this PR, the code basically looks like
clean
--- Comment #8 from rakdver at gcc dot gnu dot org 2005-12-21 14:34 ---
Gimplification lets with_cleanup_exprs escape to gimple. I am investigating
why that happens.
--
rakdver at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #7 from mmitchel at gcc dot gnu dot org 2005-12-19 18:08
---
We need to at least understand this problem before release: P1.
--
mmitchel at gcc dot gnu dot org changed:
What|Removed |Added
--
--- Comment #6 from reichelt at igpm dot rwth-aachen dot de 2005-11-29
18:44 ---
Subject: Re: [4.0/4.1/4.2 Regression] ICE on throw code
On 29 Nov, janis at gcc dot gnu dot org wrote:
> Andrew, are you sure about 3.4.0 crashing for this testcase? I tried mainline
> as far back as 200
--- Comment #5 from janis at gcc dot gnu dot org 2005-11-29 18:37 ---
The question in my previous comment should have been to Volker, not Andrew.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=24996
--- Comment #4 from janis at gcc dot gnu dot org 2005-11-29 18:36 ---
A regression hunt identified the following patch (not terribly useful):
r81764 | dnovillo | 2004-05-13 06:41:07 + (Thu, 13 May 2004) | 3 lines
Merge tree-ssa-20020619-branch into mainline.
Andrew, are you su
--- Comment #3 from reichelt at gcc dot gnu dot org 2005-11-23 17:37
---
Testcase with simpler class hierarchy:
==
struct A
{
~A();
};
struct B
{
B(A);
};
void foo(bool b)
{
throw b ? B(A()) : B(A());
}
=
--- Comment #2 from pinskia at gcc dot gnu dot org 2005-11-23 05:09 ---
Reduced testcase:
struct allocator {
~allocator();
};
struct string {
string(int, const allocator& __a = allocator());
};
struct logic_error {
logic_error(const string& __arg);
};
void foo(bool b) {
throw b ?
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
GCC build triplet|i686-pc-linux-gnu |
GCC host triplet|i686-pc-linux-gnu |
Target Milestone
--- Comment #1 from pinskia at gcc dot gnu dot org 2005-11-23 04:18 ---
Reducing.
--
pinskia at gcc dot gnu dot org changed:
What|Removed |Added
Keywords|
27 matches
Mail list logo
