* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
@ 2015-03-03 16:02 ` burnus at gcc dot gnu.org
2015-03-04 14:57 ` rguenth at gcc dot gnu.org
` (5 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: burnus at gcc dot gnu.org @ 2015-03-03 16:02 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
--- Comment #1 from Tobias Burnus <burnus at gcc dot gnu.org> ---
Created attachment 34939
--> https://gcc.gnu.org/bugzilla/attachment.cgi?id=34939&action=edit
two.ii
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
2015-03-03 16:02 ` [Bug lto/65302] " burnus at gcc dot gnu.org
@ 2015-03-04 14:57 ` rguenth at gcc dot gnu.org
2015-03-04 20:59 ` aldyh at gcc dot gnu.org
` (4 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: rguenth at gcc dot gnu.org @ 2015-03-04 14:57 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
Richard Biener <rguenth at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Target Milestone|--- |5.0
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
2015-03-03 16:02 ` [Bug lto/65302] " burnus at gcc dot gnu.org
2015-03-04 14:57 ` rguenth at gcc dot gnu.org
@ 2015-03-04 20:59 ` aldyh at gcc dot gnu.org
2015-03-05 0:00 ` hubicka at gcc dot gnu.org
` (3 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: aldyh at gcc dot gnu.org @ 2015-03-04 20:59 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
--- Comment #3 from Aldy Hernandez <aldyh at gcc dot gnu.org> ---
Started with r218767:
commit 9e7bd3ae5774b5b9d383588f42d8edab0003a9bf
Author: hubicka <hubicka@138bc75d-0d04-0410-961f-82ee72b054a4>
Date: Mon Dec 15 22:35:20 2014 +0000
PR lto/64043
* gcc.dg/lto/20110201-1_0.c: New testcase.
* tree-streamer.c (preload_common_nodes): Skip preloading
of main_identifier_node, pid_type and optimization/option nodes.
git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@218767
138bc75d-0d04-0410-96
1f-82ee72b054a4
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
` (2 preceding siblings ...)
2015-03-04 20:59 ` aldyh at gcc dot gnu.org
@ 2015-03-05 0:00 ` hubicka at gcc dot gnu.org
2015-03-06 6:44 ` hubicka at gcc dot gnu.org
` (2 subsequent siblings)
6 siblings, 0 replies; 8+ messages in thread
From: hubicka at gcc dot gnu.org @ 2015-03-05 0:00 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
Jan Hubicka <hubicka at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|NEW |ASSIGNED
Assignee|unassigned at gcc dot gnu.org |hubicka at gcc dot gnu.org
--- Comment #4 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
*free_cfg_annotations is bogus name for pass_fixup_cfg, but it is precisely
what the pass should do - clear EH edges if they are no longer needed.
I will take a look.
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
` (3 preceding siblings ...)
2015-03-05 0:00 ` hubicka at gcc dot gnu.org
@ 2015-03-06 6:44 ` hubicka at gcc dot gnu.org
2015-03-06 16:13 ` hubicka at gcc dot gnu.org
2015-03-06 16:14 ` hubicka at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: hubicka at gcc dot gnu.org @ 2015-03-06 6:44 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
--- Comment #5 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
BB 5 is:
<bb 5>:
getStringBounds (_4, _11, &D.4106, _9, _8, _7);
goto <bb 7>;
and fixup cfg calls:
8702 if (maybe_clean_eh_stmt (stmt)
8703 && gimple_purge_dead_eh_edges (bb))
the statement is correctly identified as nonthrowing but edges are removed,
because it is not in EH region.
It is created by:
#0 0x0000000000921eb5 in gimple_copy (stmt=0x7ffff68e0e58) at
../../gcc/gimple.c:1687
#1 0x0000000000f96735 in gimple_ic (icall_stmt=0x7ffff68e0e58,
direct_call=0x7ffff68e6498, prob=8000, count=0, all=0) at
../../gcc/value-prof.c:1439
#2 0x00000000007976f2 in cgraph_edge::redirect_call_stmt_to_callee
(this=0x7ffff6adf7b8) at ../../gcc/cgraph.c:1336
#3 0x000000000155a53a in inline_transform (node=0x7ffff6ae57a8) at
../../gcc/ipa-inline-transform.c:529
later the EH is copied but only if stmt can throw:
/* Build an EH edge for the direct call if necessary. */
lp_nr = lookup_stmt_eh_lp (icall_stmt);
if (lp_nr > 0 && stmt_could_throw_p (dcall_stmt))
{
add_stmt_to_eh_lp (dcall_stmt, lp_nr);
}
I suppose we want to purge EH edges in case we did not copy the EH region.
Index: value-prof.c
===================================================================
--- value-prof.c (revision 221223)
+++ value-prof.c (working copy)
@@ -1576,6 +1576,8 @@ gimple_ic (gcall *icall_stmt, struct cgr
PHI_ARG_DEF_FROM_EDGE (phi, e_eh));
}
}
+ if (!stmt_could_throw_p (dcall_stmt))
+ gimple_purge_dead_eh_edges (dcall_bb);
return dcall_stmt;
}
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
` (4 preceding siblings ...)
2015-03-06 6:44 ` hubicka at gcc dot gnu.org
@ 2015-03-06 16:13 ` hubicka at gcc dot gnu.org
2015-03-06 16:14 ` hubicka at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: hubicka at gcc dot gnu.org @ 2015-03-06 16:13 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
--- Comment #6 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
Author: hubicka
Date: Fri Mar 6 16:12:43 2015
New Revision: 221244
URL: https://gcc.gnu.org/viewcvs?rev=221244&root=gcc&view=rev
Log:
PR ipa/65302
* value-prof.c (gimple_ic): Pure dead eh edges when needed.
* g++.dg/lto/pr65302_1.C: New testcase.
* g++.dg/lto/pr65302_0.C: New testcase.
Added:
trunk/gcc/testsuite/g++.dg/lto/pr65302_0.C
trunk/gcc/testsuite/g++.dg/lto/pr65302_1.C
Modified:
trunk/gcc/ChangeLog
trunk/gcc/testsuite/ChangeLog
trunk/gcc/value-prof.c
^ permalink raw reply [flat|nested] 8+ messages in thread
* [Bug lto/65302] [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed
2015-03-03 16:02 [Bug lto/65302] New: [5 Regression] LTO: ICE internal compiler error: verify_flow_info failed burnus at gcc dot gnu.org
` (5 preceding siblings ...)
2015-03-06 16:13 ` hubicka at gcc dot gnu.org
@ 2015-03-06 16:14 ` hubicka at gcc dot gnu.org
6 siblings, 0 replies; 8+ messages in thread
From: hubicka at gcc dot gnu.org @ 2015-03-06 16:14 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65302
Jan Hubicka <hubicka at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Status|ASSIGNED |RESOLVED
Resolution|--- |FIXED
--- Comment #7 from Jan Hubicka <hubicka at gcc dot gnu.org> ---
Fixed.
^ permalink raw reply [flat|nested] 8+ messages in thread