> Hi,
> 
> This patch updates node's inline summary after edge_summary is
> updated. Otherwise it could lead to incorrect inline summary.
> 
> Bootstrapped and gcc regression test on-going.
> 
> OK for trunk?
> 
> Thanks,
> Dehao
> 
> gcc/ChangeLog:
> 
> 2014-03-21  Dehao Chen  <de...@google.com>
> 
> *ipa-inline.c (early_inliner): updates overall summary.

Looks resonable, do you have testcase where it would make a difference?

Honza
> 
> Index: gcc/ipa-inline.c
> ===================================================================
> --- gcc/ipa-inline.c (revision 208755)
> +++ gcc/ipa-inline.c (working copy)
> @@ -2318,6 +2318,7 @@ early_inliner (void)
>        edge->call_stmt, edge->callee->decl, false))
>   edge->call_stmt_cannot_inline_p = true;
>      }
> +  inline_update_overall_summary (node);
>    timevar_pop (TV_INTEGRATION);
>    iterations++;
>    inlined = false;

Reply via email to