From 5ef0eaf5dbc941b9a3ff3ec554f972657b95ed77 Mon Sep 17 00:00:00 2001 From: marxin Date: Mon, 2 Jul 2018 07:04:01 +0000 Subject: [PATCH] Revert one ipa_call_summaries::get to get_create (PR ipa/86323). 2018-07-02 Martin Liska PR ipa/86323 * ipa-inline.c (early_inliner): Revert wrongly added ::get call. 2018-07-02 Martin Liska PR ipa/86323 * g++.dg/ipa/pr86323.C: New test. git-svn-id: svn+ssh://gcc.gnu.org/svn/gcc/trunk@262297 138bc75d-0d04-0410-961f-82ee72b054a4 --- gcc/ChangeLog | 5 +++++ gcc/ipa-inline.c | 13 +++++-------- gcc/testsuite/ChangeLog | 5 +++++ gcc/testsuite/g++.dg/ipa/pr86323.C | 28 ++++++++++++++++++++++++++++ 4 files changed, 43 insertions(+), 8 deletions(-) create mode 100644 gcc/testsuite/g++.dg/ipa/pr86323.C diff --git a/gcc/ChangeLog b/gcc/ChangeLog index 64ed2f7e954..bc9edec6247 100644 --- a/gcc/ChangeLog +++ b/gcc/ChangeLog @@ -1,3 +1,8 @@ +2018-07-02 Martin Liska + + PR ipa/86323 + * ipa-inline.c (early_inliner): Revert wrongly added ::get call. + 2018-07-02 David Malcolm * dumpfile.c (dump_generic_expr_loc): Undo removal of this diff --git a/gcc/ipa-inline.c b/gcc/ipa-inline.c index a62c1ffd3b0..a84d1d9ad3e 100644 --- a/gcc/ipa-inline.c +++ b/gcc/ipa-inline.c @@ -2802,14 +2802,11 @@ early_inliner (function *fun) for (edge = node->callees; edge; edge = edge->next_callee) { /* We have no summary for new bound store calls yet. */ - ipa_call_summary *es = ipa_call_summaries->get (edge); - if (es != NULL) - { - es->call_stmt_size - = estimate_num_insns (edge->call_stmt, &eni_size_weights); - es->call_stmt_time - = estimate_num_insns (edge->call_stmt, &eni_time_weights); - } + ipa_call_summary *es = ipa_call_summaries->get_create (edge); + es->call_stmt_size + = estimate_num_insns (edge->call_stmt, &eni_size_weights); + es->call_stmt_time + = estimate_num_insns (edge->call_stmt, &eni_time_weights); if (edge->callee->decl && !gimple_check_call_matching_types ( diff --git a/gcc/testsuite/ChangeLog b/gcc/testsuite/ChangeLog index f8b82b5984b..967956fd86e 100644 --- a/gcc/testsuite/ChangeLog +++ b/gcc/testsuite/ChangeLog @@ -1,3 +1,8 @@ +2018-07-02 Martin Liska + + PR ipa/86323 + * g++.dg/ipa/pr86323.C: New test. + 2018-07-01 Tom de Vries revert: diff --git a/gcc/testsuite/g++.dg/ipa/pr86323.C b/gcc/testsuite/g++.dg/ipa/pr86323.C new file mode 100644 index 00000000000..6632f35d86c --- /dev/null +++ b/gcc/testsuite/g++.dg/ipa/pr86323.C @@ -0,0 +1,28 @@ +/* { dg-do compile } */ +/* { dg-options "-O3 --param max-early-inliner-iterations=5" } */ + +char *s; +namespace a { +template class af { +public: + af(ae); +}; +typedef af b; +namespace ag { +class ah { +public: + void ai(b aj) { c(aj); } + virtual void c(b); +}; +class d : public ah { + void c(b); +}; +class e { + void f(bool); + void ai(b aj) { g.ai(aj); } + d g; +}; +void d::c(b) {} +void e::f(bool) { ai(s); } +} +} -- 2.11.4.GIT