ok.

David

On Tue, Jun 4, 2013 at 9:51 AM, Dehao Chen <de...@google.com> wrote:
> Patch updated to set the iteration threshold to 10 for AutoFDO.
> Performance test shows ok.
>
> OK for google-4_8 branch?
>
> Thanks,
> Dehao
>
> Index: gcc/ipa-inline.c
> ===================================================================
> --- gcc/ipa-inline.c (revision 199593)
> +++ gcc/ipa-inline.c (working copy)
> @@ -2161,8 +2161,7 @@ early_inliner (void)
>      {
>        /* We iterate incremental inlining to get trivial cases of indirect
>   inlining.  */
> -      while ((flag_auto_profile
> -      || iterations < PARAM_VALUE (PARAM_EARLY_INLINER_MAX_ITERATIONS))
> +      while (iterations < PARAM_VALUE (PARAM_EARLY_INLINER_MAX_ITERATIONS)
>       && early_inline_small_functions (node))
>   {
>    timevar_push (TV_INTEGRATION);
> Index: gcc/opts.c
> ===================================================================
> --- gcc/opts.c (revision 199593)
> +++ gcc/opts.c (working copy)
> @@ -784,6 +784,10 @@ finish_options (struct gcc_options *opts, struct g
>       opts->x_param_values, opts_set->x_param_values);
>      }
>
> +  if (opts->x_flag_auto_profile)
> +    maybe_set_param_value (PARAM_EARLY_INLINER_MAX_ITERATIONS, 10,
> +   opts->x_param_values, opts_set->x_param_values);
> +
>    if (opts->x_flag_lto)
>      {
>  #ifdef ENABLE_LTO

Reply via email to