--- Comment #8 from jakub at gcc dot gnu dot org 2008-01-29 23:22 ---
Fixed.
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
Status|ASSIGNED
--- Comment #7 from jakub at gcc dot gnu dot org 2008-01-29 23:22 ---
Subject: Bug 34969
Author: jakub
Date: Tue Jan 29 23:21:24 2008
New Revision: 131946
URL: http://gcc.gnu.org/viewcvs?root=gcc&view=rev&rev=131946
Log:
PR middle-end/34969
* cgraph.h (cgraph_update_edg
--- Comment #6 from hubicka at ucw dot cz 2008-01-28 20:51 ---
Subject: Re: [4.3 regression] ICE with -fipa-cp -ffast-math
> No, I mean providing something like cgraph_update_edges_for_call_stmt (tree
> old, tree new); or alternatively cgraph_remove_edge_from_call_stmt () and
> cgraph_
--- Comment #5 from rguenth at gcc dot gnu dot org 2008-01-28 12:39 ---
No, I mean providing something like cgraph_update_edges_for_call_stmt (tree
old, tree new); or alternatively cgraph_remove_edge_from_call_stmt () and
cgraph_add_edge_from_call_stmt () and call those two unconditional
--- Comment #4 from jakub at gcc dot gnu dot org 2008-01-28 12:34 ---
Not folding CALL_EXPRs would introduce a regression on g++.dg/opt/devirt1.C,
after all fold_marked_statements has been added exactly to fix that regression:
http://gcc.gnu.org/ml/gcc-patches/2007-01/msg00165.html
Rega
--- Comment #3 from rguenth at gcc dot gnu dot org 2008-01-28 11:11 ---
I suppose a simpler fix for 4.3 would be to not fold call stmts here.
Otherwise
moving the cgraph adjustment to a helper function in cgraph.c would look
nicer.
--
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=349
--- Comment #2 from jakub at gcc dot gnu dot org 2008-01-28 09:52 ---
Fix:
2008-01-28 Jakub Jelinek <[EMAIL PROTECTED]>
PR middle-end/34969
* tree-inline.c (fold_marked_statements): Update resp. remove
cgraph edges if a call statement has been folded.
*
--
jakub at gcc dot gnu dot org changed:
What|Removed |Added
AssignedTo|unassigned at gcc dot gnu |jakub at gcc dot gnu dot org
|dot org
--- Comment #1 from rguenth at gcc dot gnu dot org 2008-01-25 10:06 ---
We ICE in verify_cgraph because at the time we dump the call stmt here:
#5 0x00b9d3ec in verify_cgraph_node (node=0x2af3434e1e00)
at /space/rguenther/src/svn/trunk/gcc/cgraphunit.c:800
800
--
reichelt at gcc dot gnu dot org changed:
What|Removed |Added
Target Milestone|--- |4.3.0
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=34969
10 matches
Mail list logo