* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
@ 2012-10-12 9:37 ` rguenth at gcc dot gnu.org
2012-10-12 9:59 ` doko at gcc dot gnu.org
` (7 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-10-12 9:37 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |4.7.3
--- Comment #1 from Richard Biener <rguenth at gcc dot gnu.org> 2012-10-12 09:37:04 UTC ---
Please also check trunk.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
2012-10-12 9:37 ` [Bug target/54902] " rguenth at gcc dot gnu.org
@ 2012-10-12 9:59 ` doko at gcc dot gnu.org
2012-10-13 15:39 ` mikpe at it dot uu.se
` (6 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: doko at gcc dot gnu.org @ 2012-10-12 9:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
Matthias Klose <doko at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Known to work| |4.8.0
--- Comment #2 from Matthias Klose <doko at gcc dot gnu.org> 2012-10-12 09:58:46 UTC ---
trunk 20120820 and 20121008 do work.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
2012-10-12 9:37 ` [Bug target/54902] " rguenth at gcc dot gnu.org
2012-10-12 9:59 ` doko at gcc dot gnu.org
@ 2012-10-13 15:39 ` mikpe at it dot uu.se
2012-10-23 5:35 ` zhenqiang.chen at linaro dot org
` (5 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: mikpe at it dot uu.se @ 2012-10-13 15:39 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
Mikael Pettersson <mikpe at it dot uu.se> changed:
What |Removed |Added
----------------------------------------------------------------------------
CC| |mikpe at it dot uu.se
--- Comment #3 from Mikael Pettersson <mikpe at it dot uu.se> 2012-10-13 15:39:25 UTC ---
I can reproduce the ICE with gcc-4.7.2 on armv5tel-linux-gnueabi, sparc64-linux
-m32, x86_64-linux -m32, and m68k-linux. I haven't checked any other targets.
The ICE stopped on trunk with Richard B's "Make gsi_remove return whether EH
cleanup is required" patch in r186159:
http://gcc.gnu.org/ml/gcc-patches/2012-04/msg00202.html
http://gcc.gnu.org/ml/gcc-cvs/2012-04/msg00108.html
Backporting r186159 and the follow-up fix in r186164 to gcc-4.7.2 eliminates
the ICE there for the four targets mentioned above.
The description of Richard's patch makes it sound more like a cleanup than a
bug fix, so I'm not sure if it actually fixes the bug or just masks it.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
` (2 preceding siblings ...)
2012-10-13 15:39 ` mikpe at it dot uu.se
@ 2012-10-23 5:35 ` zhenqiang.chen at linaro dot org
2012-10-24 13:41 ` rguenth at gcc dot gnu.org
` (4 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: zhenqiang.chen at linaro dot org @ 2012-10-23 5:35 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
--- Comment #4 from zhenqiang.chen at linaro dot org 2012-10-23 05:35:17 UTC ---
Created attachment 28512
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28512
gdb backtrace
Root cause:
PRE bases on the result of value numbering (run_scc_vn). At the end, it
free_scc_vn. But before free_scc_vn, it might call cleanup_tree_cfg ();
if (do_eh_cleanup || do_ab_cleanup)
cleanup_tree_cfg ();
cleanup_tree_cfg might call make_ssa_name which might reuse some "name" from
the FREE_SSANAMES list. If the VN_INFO of the "name" is NULL, free_scc_vn will
"Segmentation fault".
Richard's patch makes "do_eh_cleanup" to false. So cleanup_tree_cfg is not
called in PRE. Then no new ssa name will be created.
bt.log shows the gdb backtrace to create a new ssa name.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
` (3 preceding siblings ...)
2012-10-23 5:35 ` zhenqiang.chen at linaro dot org
@ 2012-10-24 13:41 ` rguenth at gcc dot gnu.org
2012-10-24 13:59 ` rguenth at gcc dot gnu.org
` (3 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-10-24 13:41 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|UNCONFIRMED |ASSIGNED
Last reconfirmed| |2012-10-24
AssignedTo|unassigned at gcc dot |rguenth at gcc dot gnu.org
|gnu.org |
Ever Confirmed|0 |1
--- Comment #5 from Richard Biener <rguenth at gcc dot gnu.org> 2012-10-24 13:40:38 UTC ---
Mine.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
` (4 preceding siblings ...)
2012-10-24 13:41 ` rguenth at gcc dot gnu.org
@ 2012-10-24 13:59 ` rguenth at gcc dot gnu.org
2012-10-25 11:06 ` rguenth at gcc dot gnu.org
` (2 subsequent siblings)
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-10-24 13:59 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
--- Comment #6 from Richard Biener <rguenth at gcc dot gnu.org> 2012-10-24 13:58:50 UTC ---
Created attachment 28517
--> http://gcc.gnu.org/bugzilla/attachment.cgi?id=28517
patch
Patch I am testing.
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
` (5 preceding siblings ...)
2012-10-24 13:59 ` rguenth at gcc dot gnu.org
@ 2012-10-25 11:06 ` rguenth at gcc dot gnu.org
2012-10-25 11:16 ` rguenth at gcc dot gnu.org
2012-10-25 11:32 ` rguenth at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-10-25 11:06 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
--- Comment #7 from Richard Biener <rguenth at gcc dot gnu.org> 2012-10-25 11:05:49 UTC ---
Author: rguenth
Date: Thu Oct 25 11:05:43 2012
New Revision: 192803
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192803
Log:
2012-10-25 Richard Biener <rguenther@suse.de>
PR tree-optimization/54902
* tree-ssa-pre.c (fini_pre): Return TODO.
(execute_pre): Adjust.
* tree-ssa-tailmerge.c (tail_merge_optimize): Delete unreachable
blocks before computing dominators.
* g++.dg/torture/pr54902.C: New testcase.
Added:
branches/gcc-4_7-branch/gcc/testsuite/g++.dg/torture/pr54902.C
Modified:
branches/gcc-4_7-branch/gcc/ChangeLog
branches/gcc-4_7-branch/gcc/testsuite/ChangeLog
branches/gcc-4_7-branch/gcc/tree-ssa-pre.c
branches/gcc-4_7-branch/gcc/tree-ssa-tail-merge.c
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
` (6 preceding siblings ...)
2012-10-25 11:06 ` rguenth at gcc dot gnu.org
@ 2012-10-25 11:16 ` rguenth at gcc dot gnu.org
2012-10-25 11:32 ` rguenth at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-10-25 11:16 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
--- Comment #8 from Richard Biener <rguenth at gcc dot gnu.org> 2012-10-25 11:16:19 UTC ---
Author: rguenth
Date: Thu Oct 25 11:16:12 2012
New Revision: 192804
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=192804
Log:
2012-10-25 Richard Biener <rguenther@suse.de>
PR tree-optimization/54902
* tree-ssa-pre.c (fini_eliminate): Return TODO.
(do_pre): Adjust.
(execute_fre): Likewise.
* tree-ssa-tailmerge.c (tail_merge_optimize): Delete unreachable
blocks before computing dominators.
* g++.dg/torture/pr54902.C: New testcase.
Added:
trunk/gcc/testsuite/g++.dg/torture/pr54902.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/tree-ssa-pre.c
trunk/gcc/tree-ssa-tail-merge.c
^ permalink raw reply [flat|nested] 10+ messages in thread* [Bug target/54902] [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi
2012-10-11 15:34 [Bug target/54902] New: [4.7 Regression], ICE (segfault) building on arm-linux-gnueabi doko at gcc dot gnu.org
` (7 preceding siblings ...)
2012-10-25 11:16 ` rguenth at gcc dot gnu.org
@ 2012-10-25 11:32 ` rguenth at gcc dot gnu.org
8 siblings, 0 replies; 10+ messages in thread
From: rguenth at gcc dot gnu.org @ 2012-10-25 11:32 UTC (permalink / raw)
To: gcc-bugs
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=54902
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution| |FIXED
--- Comment #9 from Richard Biener <rguenth at gcc dot gnu.org> 2012-10-25 11:32:04 UTC ---
Fixed.
^ permalink raw reply [flat|nested] 10+ messages in thread