[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #20 from pinskia at gcc dot gnu dot org 2007-08-24 00:24 --- *** Bug 28358 has been marked as a duplicate of this bug. *** -- pinskia at gcc dot gnu dot org changed: What|Removed |Added CC||peter dot soetens at fmtc ||dot be http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #19 from tbm at cyrius dot com 2007-02-03 09:47 --- (In reply to comment #18) Fixed. Richi, do you think you can check whether PR28358 is really a duplicate of this bug (as Andrew thinks) and should be closed. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #14 from rguenth at gcc dot gnu dot org 2007-01-05 11:43 --- Mine. I'm testing the inliner fix. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added AssignedTo|unassigned at gcc dot gnu |rguenth at gcc dot gnu dot |dot org |org Status|NEW |ASSIGNED Last reconfirmed|2006-10-18 09:46:51 |2007-01-05 11:43:22 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #15 from rguenth at gcc dot gnu dot org 2007-01-05 13:53 --- Subject: Bug 28116 Author: rguenth Date: Fri Jan 5 13:53:45 2007 New Revision: 120480 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=120480 Log: 2007-01-05 Richard Guenther [EMAIL PROTECTED] PR middle-end/28116 * tree-inline.c (copy_body_r): Mark argument of ADDR_EXPR addressable. * g++.dg/opt/pr28116.C: New testcase. Added: branches/gcc-4_1-branch/gcc/testsuite/g++.dg/opt/pr28116.C Modified: branches/gcc-4_1-branch/gcc/ChangeLog branches/gcc-4_1-branch/gcc/testsuite/ChangeLog branches/gcc-4_1-branch/gcc/tree-inline.c -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #16 from rguenth at gcc dot gnu dot org 2007-01-05 13:57 --- Subject: Bug 28116 Author: rguenth Date: Fri Jan 5 13:56:54 2007 New Revision: 120481 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=120481 Log: 2007-01-05 Richard Guenther [EMAIL PROTECTED] PR middle-end/28116 * g++.dg/opt/pr28116.C: New testcase. Added: branches/gcc-4_2-branch/gcc/testsuite/g++.dg/opt/pr28116.C - copied unchanged from r120480, branches/gcc-4_1-branch/gcc/testsuite/g++.dg/opt/pr28116.C Modified: branches/gcc-4_2-branch/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #17 from rguenth at gcc dot gnu dot org 2007-01-05 14:00 --- Subject: Bug 28116 Author: rguenth Date: Fri Jan 5 14:00:46 2007 New Revision: 120482 URL: http://gcc.gnu.org/viewcvs?root=gccview=revrev=120482 Log: 2007-01-05 Richard Guenther [EMAIL PROTECTED] PR middle-end/28116 * g++.dg/opt/pr28116.C: New testcase. Added: trunk/gcc/testsuite/g++.dg/opt/pr28116.C - copied unchanged from r120480, branches/gcc-4_1-branch/gcc/testsuite/g++.dg/opt/pr28116.C Modified: trunk/gcc/testsuite/ChangeLog -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #18 from rguenth at gcc dot gnu dot org 2007-01-05 14:02 --- Fixed. -- rguenth at gcc dot gnu dot org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution||FIXED http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #11 from chris_clayton at f1internet dot com 2006-11-26 12:54 --- I'm the reporter of 29809, which is a duplicate of this bug. I'm not familiar with the workflow here, so do I simply wait for 4.1.2 or is there likely to be patch to test before 4.1.2 is released? Is the patch at comment 4 of bug 29809 worth testing - the author seemed to think it was too simple to be true :)? Thanks -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #12 from rguenth at gcc dot gnu dot org 2006-11-26 13:09 --- Usually you wait until a patch is available or 4.1.2 is released (supposed the problem is fixed there). You can try testing the ad-hoc fix in PR29809 comment #4. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #13 from chris_clayton at f1internet dot com 2006-11-26 21:22 --- (In reply to comment #12) Usually you wait until a patch is available or 4.1.2 is released (supposed the problem is fixed there). You can try testing the ad-hoc fix in PR29809 comment #4. OK, I've updated to the 20061124 snapshot. Without the patch at PR29809, I get the ICE building avifile-0.7.45 and konverter-0.92-beta1. With the patch, both builds complete. From my, admittedly minimal, testing, konverter seems to be working OK. Unfortunately, I don't have the hardware to test the sample application from the avifile package. Hope this helps. -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #10 from pinskia at gcc dot gnu dot org 2006-11-12 17:07 --- *** Bug 29809 has been marked as a duplicate of this bug. *** -- pinskia at gcc dot gnu dot org changed: What|Removed |Added CC||chris_clayton at f1internet ||dot com http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
-- mmitchel at gcc dot gnu dot org changed: What|Removed |Added Priority|P3 |P1 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #9 from janis at gcc dot gnu dot org 2006-10-19 16:35 --- I wasn't able to do a regression hunt for the fix on the mainline because I didn't find any revision for which the test failed on the mainline. A regression hunt on the 4.1 branch using the testcase from comment #7 with an i686-linux cross compiler identified this patch where the test starts getting the ICE: http://gcc.gnu.org/viewcvs?view=revrev=110838 r110838 | jason | 2006-02-10 17:32:10 + (Fri, 10 Feb 2006) The mainline didn't ICE when that patch was added. -- janis at gcc dot gnu dot org changed: What|Removed |Added CC||jason at gcc dot gnu dot org http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #8 from rguenth at gcc dot gnu dot org 2006-10-18 09:46 --- Janis, can you check what fixed this on the mainline? Thanks! -- rguenth at gcc dot gnu dot org changed: What|Removed |Added CC||janis at gcc dot gnu dot org Known to fail|4.1.1 |4.1.0 4.1.1 Last reconfirmed|2006-06-21 12:20:43 |2006-10-18 09:46:51 date|| http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #6 from pinskia at gcc dot gnu dot org 2006-08-25 06:05 --- *** Bug 28840 has been marked as a duplicate of this bug. *** -- pinskia at gcc dot gnu dot org changed: What|Removed |Added CC||eemax at web dot de http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #7 from bonzini at gnu dot org 2006-08-25 06:33 ---
reduced testcase from pr28840
struct QDateTime
{
QDateTime addSecs( int secs ) const;
int t;
};
QDateTime gridToDate(long x)
{
QDateTime date;
date = date.addSecs(1);
return date;
}
void whatsOnAt(long x, long y)
{
gridToDate(x);
}
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #5 from jakub at gcc dot gnu dot org 2006-06-22 16:58 --- In GCC trunk bootstrap + regression testing the tree-nrv optimization hits 584 types, out of this 38 cases have TREE_ADDRESSABLE set on the RESULT_DECL (because C++ FE already NRV optimized it, address of retval was taken usually when it was constructed, then some optimizations later on set retval to a temporary variable and the code that takes result was optimized out), e.g. when compiling fstream-inst.cc in libstdc++-v3. So, bailing out in tree_nrv whenever TREE_ADDRESSABLE (result) would pessimize those. On the other side, in tree_nrv we know that TREE_ADDRESSABLE (found) == 0 (we check for that), so we can either stop copying the addressable flag, or bail if TREE_ADDRESSABLE (result). -- http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
--- Comment #4 from jakub at gcc dot gnu dot org 2006-06-21 12:20 ---
This sounds like a tree-nrv bug to me.
We have at *.optimized:
QTime QTime::msToTime() (this)
{
struct QTimeD.1732 D.1772;
struct QTimeD.1732 tD.1765;
(voidD.34) 0;
D.1772 = addMSecs (retval);
retval = D.1772;
return retval;
}
(note D.1772 never has address taken, while retval does).
Now tree_nrv () finds retval is only set to D.1772 and:
/* At this point we know that all the return statements return the
same local which has suitable attributes for NRV. Copy debugging
information from FOUND to RESULT. */
DECL_NAME (result) = DECL_NAME (found);
DECL_SOURCE_LOCATION (result) = DECL_SOURCE_LOCATION (found);
DECL_ABSTRACT_ORIGIN (result) = DECL_ABSTRACT_ORIGIN (found);
TREE_ADDRESSABLE (result) = TREE_ADDRESSABLE (found);
which clears the addressable flag, although the RESULT_DECL clearly has its
address taken. Perhaps the last line should be:
TREE_ADDRESSABLE (result) |= TREE_ADDRESSABLE (found);
but am not 100% sure if tree_nrv is a safe transformation if TREE_ADDRESSABLE
(result) was set already. Also, in this case DECL_NAME (result) contained the
real var name (t) while DECL_NAME (found) is NULL, so perhaps the first 3 lines
should be conditionalized on if (DECL_NAME (found)) to avoid making debug info
worse.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
CC||dnovillo at redhat dot com,
||law at gcc dot gnu dot org
Status|UNCONFIRMED |NEW
Ever Confirmed|0 |1
Last reconfirmed|-00-00 00:00:00 |2006-06-21 12:20:43
date||
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
[Bug middle-end/28116] [4.1 Regression] ICE when building konverter with gcc-4.1 with -O3 [RSO]
-- pinskia at gcc dot gnu dot org changed: What|Removed |Added Component|c++ |middle-end Summary|[4.1 Regression] ICE when |[4.1 Regression] ICE when |building konverter with gcc-|building konverter with gcc- |4.1 with -O3|4.1 with -O3 [RSO] Target Milestone|--- |4.1.2 http://gcc.gnu.org/bugzilla/show_bug.cgi?id=28116
