From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: by sourceware.org (Postfix, from userid 1611) id 02F74385B835; Thu, 16 Apr 2020 17:21:34 +0000 (GMT) DKIM-Filter: OpenDKIM Filter v2.11.0 sourceware.org 02F74385B835 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gcc.gnu.org; s=default; t=1587057695; bh=7Rx4iVKeJJtS/zvOPp5rd0kfq9+WUOs2pUrQgPOrcAA=; h=From:To:Subject:Date:From; b=pk3vlV3gh+fjcaV9NqZeo4IRtDxWFdeMXXEtRXXRdUWKBT048sCbZH7eZhOKKJ02D ELEqcDiAlJXlzNyRRcjRELPNuYHAHPw84ciMLTn48E2hpe69A5YZmI/mPApN0OU2cE +Q2kAKfrU5FnSpLPrOTYbO9VZyHIX+jvwDWn2Cho= Content-Type: text/plain; charset="us-ascii" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit From: Martin Jambor To: gcc-cvs@gcc.gnu.org Subject: [gcc r10-7760] ipa: Make call redirection detect already adjusted calls (PR 93621) X-Act-Checkin: gcc X-Git-Author: Martin Jambor X-Git-Refname: refs/heads/master X-Git-Oldrev: 26bebf576ddcdcfb596f07e8c2896f17c48516e7 X-Git-Newrev: 7123347c8b44571811c4b58506b06fb09969bccb Message-Id: <20200416172135.02F74385B835@sourceware.org> Date: Thu, 16 Apr 2020 17:21:34 +0000 (GMT) X-BeenThere: gcc-cvs@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-cvs mailing list List-Unsubscribe: , List-Archive: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 16 Apr 2020 17:21:35 -0000 https://gcc.gnu.org/g:7123347c8b44571811c4b58506b06fb09969bccb commit r10-7760-g7123347c8b44571811c4b58506b06fb09969bccb Author: Martin Jambor Date: Thu Apr 16 19:21:02 2020 +0200 ipa: Make call redirection detect already adjusted calls (PR 93621) PR 93621 testcase makes redirect_call_stmt_to_callee wrongly assume that a call statement needs redirecting but then rightly fails an assert ensuring the call statement parameters have not already been adjusted because they were already created adjusted as part of thunk expansion. The test fails because the decl in the call call statement is different than the decl of the callee, because the latter was created in save_inline_function_body. This patch adds a way to link these two and detect the situation in redirect_call_stmt_to_callee. 2020-04-16 Martin Jambor PR ipa/93621 * ipa-inline.h (ipa_saved_clone_sources): Declare. * ipa-inline-transform.c (ipa_saved_clone_sources): New variable. (save_inline_function_body): Link the new body holder with the previous one. * cgraph.c: Include ipa-inline.h. (cgraph_edge::redirect_call_stmt_to_callee): Try to find the decl from the statement in ipa_saved_clone_sources. * cgraphunit.c: Include ipa-inline.h. (expand_all_functions): Free ipa_saved_clone_sources. testsuite/ * g++.dg/ipa/pr93621.C: New test. Diff: --- gcc/ChangeLog | 13 +++++++++++++ gcc/cgraph.c | 11 +++++++++++ gcc/cgraphunit.c | 4 +++- gcc/ipa-inline-transform.c | 19 +++++++++++++++++++ gcc/ipa-inline.h | 1 + gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/g++.dg/ipa/pr93621.C | 29 +++++++++++++++++++++++++++++ 7 files changed, 81 insertions(+), 1 deletion(-) diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 0c7dad08637..5fd869a9239 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,16 @@ +2020-04-16 Martin Jambor + + PR ipa/93621 + * ipa-inline.h (ipa_saved_clone_sources): Declare. + * ipa-inline-transform.c (ipa_saved_clone_sources): New variable. + (save_inline_function_body): Link the new body holder with the + previous one. + * cgraph.c: Include ipa-inline.h. + (cgraph_edge::redirect_call_stmt_to_callee): Try to find the decl from + the statement in ipa_saved_clone_sources. + * cgraphunit.c: Include ipa-inline.h. + (expand_all_functions): Free ipa_saved_clone_sources. + 2020-04-16 Richard Sandiford PR target/94606 diff --git a/gcc/cgraph.c b/gcc/cgraph.c index ecb234d032f..72d7cb54301 100644 --- a/gcc/cgraph.c +++ b/gcc/cgraph.c @@ -63,6 +63,7 @@ along with GCC; see the file COPYING3. If not see #include "attribs.h" #include "selftest.h" #include "tree-into-ssa.h" +#include "ipa-inline.h" /* FIXME: Only for PROP_loops, but cgraph shouldn't have to know about this. */ #include "tree-pass.h" @@ -1470,6 +1471,16 @@ cgraph_edge::redirect_call_stmt_to_callee (cgraph_edge *e) || decl == e->callee->decl) return e->call_stmt; + if (decl && ipa_saved_clone_sources) + { + tree *p = ipa_saved_clone_sources->get (e->callee); + if (p && decl == *p) + { + gimple_call_set_fndecl (e->call_stmt, e->callee->decl); + return e->call_stmt; + } + } + if (flag_checking && decl) { cgraph_node *node = cgraph_node::get (decl); diff --git a/gcc/cgraphunit.c b/gcc/cgraphunit.c index 0e255f25b7d..a1ace95879a 100644 --- a/gcc/cgraphunit.c +++ b/gcc/cgraphunit.c @@ -205,6 +205,7 @@ along with GCC; see the file COPYING3. If not see #include "lto-section-names.h" #include "stringpool.h" #include "attribs.h" +#include "ipa-inline.h" /* Queue of cgraph nodes scheduled to be added into cgraph. This is a secondary queue used during optimization to accommodate passes that @@ -2481,7 +2482,8 @@ expand_all_functions (void) symtab->process_new_functions (); free_gimplify_stack (); - + delete ipa_saved_clone_sources; + ipa_saved_clone_sources = NULL; free (order); } diff --git a/gcc/ipa-inline-transform.c b/gcc/ipa-inline-transform.c index eed992d314d..be60bbccb5c 100644 --- a/gcc/ipa-inline-transform.c +++ b/gcc/ipa-inline-transform.c @@ -531,6 +531,11 @@ inline_call (struct cgraph_edge *e, bool update_original, return new_edges_found; } +/* For each node that was made the holder of function body by + save_inline_function_body, this summary contains pointer to the previous + holder of the body. */ + +function_summary *ipa_saved_clone_sources; /* Copy function body of NODE and redirect all inline clones to it. This is done before inline plan is applied to NODE when there are @@ -588,6 +593,20 @@ save_inline_function_body (struct cgraph_node *node) first_clone->next_sibling_clone = NULL; gcc_assert (!first_clone->prev_sibling_clone); } + + tree prev_body_holder = node->decl; + if (!ipa_saved_clone_sources) + ipa_saved_clone_sources = new function_summary (symtab); + else + { + tree *p = ipa_saved_clone_sources->get (node); + if (p) + { + prev_body_holder = *p; + gcc_assert (prev_body_holder); + } + } + *ipa_saved_clone_sources->get_create (first_clone) = prev_body_holder; first_clone->clone_of = NULL; /* Now node in question has no clones. */ diff --git a/gcc/ipa-inline.h b/gcc/ipa-inline.h index 5025b6045fc..74c08685e49 100644 --- a/gcc/ipa-inline.h +++ b/gcc/ipa-inline.h @@ -65,6 +65,7 @@ void clone_inlined_nodes (struct cgraph_edge *e, bool, bool, int *); extern int ncalls_inlined; extern int nfunctions_inlined; +extern function_summary *ipa_saved_clone_sources; /* Return estimated size of the inline sequence of EDGE. */ diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index 72bed4064e5..ad4b7e29ec7 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2020-04-16 Martin Jambor + + PR ipa/93621 + * g++.dg/ipa/pr93621.C: New test. + 2020-04-16 Richard Sandiford PR target/94606 diff --git a/gcc/testsuite/g++.dg/ipa/pr93621.C b/gcc/testsuite/g++.dg/ipa/pr93621.C new file mode 100644 index 00000000000..ffe6bbdae2e --- /dev/null +++ b/gcc/testsuite/g++.dg/ipa/pr93621.C @@ -0,0 +1,29 @@ +// PR ipa/93621 +// { dg-do compile } +// { dg-options "-O3 --param ipa-cp-eval-threshold=100 --param large-function-growth=60 --param large-function-insns=10 --param uninlined-thunk-insns=1000" } + +typedef enum { X } E; +struct A { + virtual void bar (); +}; +struct B { + virtual E fn (const char *, int, int *) = 0; +}; +struct C : A, B { + E fn (const char *, int, int *); + void fn2 (); + B *foo; +}; +void C::fn2 () { + if (!foo) + return; + foo->fn (0, 0, 0); +} +E +C::fn (const char *, int, int *) +{ + fn2 (); + foo = 0; + fn (0, 0, 0); + return X; +}