diff mbox

[GOOGLE] call compute_inline_parameters before early_inliner

Message ID CAO2gOZUPwv7TM7wKc=_5kD5uKnyDVWwXbqu3WRgSU_vsSZtMRw@mail.gmail.com
State New
Headers show

Commit Message

Dehao Chen Feb. 26, 2014, 11:23 p.m. UTC 
This patch fixes the bug of not calling compute_inline_parameters
before early_inliner, which would lead to ICE.

Testing on going, OK for google-4_8 if test passes?

Thanks,
Dehao

Comments

Xinliang David Li Feb. 26, 2014, 11:25 p.m. UTC | #1 
On Wed, Feb 26, 2014 at 3:23 PM, Dehao Chen <dehao@google.com> wrote:
> This patch fixes the bug of not calling compute_inline_parameters
> before early_inliner, which would lead to ICE.
>
> Testing on going, OK for google-4_8 if test passes?
>
> Thanks,
> Dehao
>
> Index: gcc/auto-profile.c
> ===================================================================
> --- gcc/auto-profile.c (revision 207970)
> +++ gcc/auto-profile.c (working copy)
> @@ -1533,6 +1533,8 @@ auto_profile (void)
>    early_inliner ();
>   }
>
> +      compute_inline_parameters (cgraph_get_node (current_function_decl),
> + false);

false --> true ?

David

>        early_inliner ();
>        autofdo::afdo_annotate_cfg (promoted_stmts);
>        compute_function_frequency ();
diff mbox

Patch

Index: gcc/auto-profile.c
===================================================================
--- gcc/auto-profile.c (revision 207970)
+++ gcc/auto-profile.c (working copy)
@@ -1533,6 +1533,8 @@  auto_profile (void)
   early_inliner ();
  }

+      compute_inline_parameters (cgraph_get_node (current_function_decl),
+ false);
       early_inliner ();
       autofdo::afdo_annotate_cfg (promoted_stmts);
       compute_function_frequency ();