From d83fa499b9895618ae8f63c004f3090b71d488ae Mon Sep 17 00:00:00 2001 From: Eric Botcazou Date: Fri, 17 Oct 2014 19:53:18 +0000 Subject: [PATCH] ipa-inline-transform.c (master_clone_with_noninline_clones_p): New. * ipa-inline-transform.c (master_clone_with_noninline_clones_p): New. (clone_inlined_nodes): Do not overwrite the clone if above predicate returns true. From-SVN: r216415 --- gcc/ChangeLog | 6 ++++++ gcc/ipa-inline-transform.c | 19 ++++++++++++++++++- 2 files changed, 24 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index af39978c997..b2a8e7a2a6f 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,9 @@ +2014-10-17 Eric Botcazou + + * ipa-inline-transform.c (master_clone_with_noninline_clones_p): New. + (clone_inlined_nodes): Do not overwrite the clone if above predicate + returns true. + 2014-10-17 Ilya Tocar * config/i386/i386.c (MAX_VECT_LEN): Move earlier. diff --git a/gcc/ipa-inline-transform.c b/gcc/ipa-inline-transform.c index 4b0fcad396f..1cb4c0520e8 100644 --- a/gcc/ipa-inline-transform.c +++ b/gcc/ipa-inline-transform.c @@ -122,6 +122,20 @@ can_remove_node_now_p (struct cgraph_node *node, struct cgraph_edge *e) return true; } +/* Return true if NODE is a master clone with non-inline clones. */ + +static bool +master_clone_with_noninline_clones_p (struct cgraph_node *node) +{ + if (node->clone_of) + return false; + + for (struct cgraph_node *n = node->clones; n; n = n->next_sibling_clone) + if (n->decl != node->decl) + return true; + + return false; +} /* E is expected to be an edge being inlined. Clone destination node of the edge and redirect it to the new clone. @@ -155,7 +169,10 @@ clone_inlined_nodes (struct cgraph_edge *e, bool duplicate, /* Recursive inlining never wants the master clone to be overwritten. */ && update_original - && can_remove_node_now_p (e->callee, e)) + && can_remove_node_now_p (e->callee, e) + /* We cannot overwrite a master clone with non-inline clones + until after these clones are materialized. */ + && !master_clone_with_noninline_clones_p (e->callee)) { /* TODO: When callee is in a comdat group, we could remove all of it, including all inline clones inlined into it. That would however -- 2.30.2