[Bug tree-optimization/57539] [4.9 Regression] ice in ipa_edge_duplication_hook
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57539 Martin Jambor jamborm at gcc dot gnu.org changed: What|Removed |Added Status|ASSIGNED|RESOLVED Resolution|--- |FIXED --- Comment #5 from Martin Jambor jamborm at gcc dot gnu.org --- Author: jamborm Date: Mon Jun 24 12:40:17 2013 New Revision: 200368 URL: http://gcc.gnu.org/viewcvs?rev=200368root=gccview=rev Log: 2013-06-24 Martin Jambor mjam...@suse.cz PR tree-optimization/57539 * cgraphclones.c (cgraph_clone_node): Add parameter new_inlined_to, set global.inlined_to of the new node to it. All callers changed. * ipa-inline-transform.c (clone_inlined_nodes): New variable inlining_into, pass it to cgraph_clone_node. * ipa-prop.c (ipa_propagate_indirect_call_infos): Do not call ipa_free_edge_args_substructures. (ipa_edge_duplication_hook): Only add edges from inlined nodes to rdesc linked list. Do not assert rdesc edges have inlined caller. Assert we have found an rdesc in the rdesc list. Modified: trunk/gcc/ChangeLog trunk/gcc/cgraph.h trunk/gcc/cgraphclones.c trunk/gcc/ipa-inline-transform.c trunk/gcc/ipa-inline.c trunk/gcc/ipa-prop.c trunk/gcc/lto-cgraph.c
[Bug tree-optimization/57539] [4.9 Regression] ice in ipa_edge_duplication_hook
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57539 Martin Jambor jamborm at gcc dot gnu.org changed: What|Removed |Added URL||http://gcc.gnu.org/ml/gcc-p ||atches/2013-06/msg00661.htm ||l --- Comment #4 from Martin Jambor jamborm at gcc dot gnu.org --- I have submitted the fix to the mailing list: http://gcc.gnu.org/ml/gcc-patches/2013-06/msg00661.html
[Bug tree-optimization/57539] [4.9 Regression] ice in ipa_edge_duplication_hook
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57539 --- Comment #3 from Martin Jambor jamborm at gcc dot gnu.org --- Created attachment 30286 -- http://gcc.gnu.org/bugzilla/attachment.cgi?id=30286action=edit Proposed fix I'm currently bootstrapping and testing this patch to fix the issue. I'll give one more thought to creating a testcase for the testsuite but constructing one is not entirely trivial.
[Bug tree-optimization/57539] [4.9 Regression] ice in ipa_edge_duplication_hook
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57539 Martin Jambor jamborm at gcc dot gnu.org changed: What|Removed |Added CC||hubicka at gcc dot gnu.org --- Comment #2 from Martin Jambor jamborm at gcc dot gnu.org --- The main reason why the code does not work is that when ipa_edge_duplication_hook is invoked, dst-caller does not yet have its global.inlined_to set (it does not have any callers either) and so we cannot use it to figure out which copy of rdesc in the list is the one describing the new tree of inline clones. This means that this whole copying/re-mapping should be moved out of the hook into a specialized function called from clone_inlined_nodes. Additionally, and this has confused me for hours, we also do not even create the rdesc we want to map a different edge to in this particular testcase. That is because an originally indirect edge is involved and the call to ipa_free_edge_args_substructures in ipa_propagate_indirect_call_infos. That deallocated the vector of jump functions, so the loop over it in the hook did not do anything. That call needs to be removed.
[Bug tree-optimization/57539] [4.9 Regression] ice in ipa_edge_duplication_hook
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57539 Richard Biener rguenth at gcc dot gnu.org changed: What|Removed |Added CC||mjambor at suse dot cz Component|c |tree-optimization Target Milestone|--- |4.9.0 Summary|ice in |[4.9 Regression] ice in |ipa_edge_duplication_hook |ipa_edge_duplication_hook
[Bug tree-optimization/57539] [4.9 Regression] ice in ipa_edge_duplication_hook
http://gcc.gnu.org/bugzilla/show_bug.cgi?id=57539 Martin Jambor jamborm at gcc dot gnu.org changed: What|Removed |Added Status|UNCONFIRMED |ASSIGNED Last reconfirmed||2013-06-06 CC|mjambor at suse dot cz |jamborm at gcc dot gnu.org Assignee|unassigned at gcc dot gnu.org |jamborm at gcc dot gnu.org Ever confirmed|0 |1 --- Comment #1 from Martin Jambor jamborm at gcc dot gnu.org --- Confirmed, mine.