Patchwork [GOOGLE] max-lipo-group parameter for AutoFDO

login
register
mail settings
Submitter Dehao Chen
Date Sept. 26, 2013, 4 p.m.
Message ID <CAO2gOZWShrynMHsCseqqjjJoTCXi8cNjXzBxicGU++OOV+zuDw@mail.gmail.com>
Download mbox | patch
Permalink /patch/278218/
State New
Headers show

Comments

Dehao Chen - Sept. 26, 2013, 4 p.m.
This patch fix the bug when setting max-lipo-group in AutoFDO:

Bootstrapped and passed regression test.

OK for google branches?

Thanks,
Dehao
Xinliang David Li - Sept. 26, 2013, 4:03 p.m.
Looks ok.

David

On Thu, Sep 26, 2013 at 9:00 AM, Dehao Chen <dehao@google.com> wrote:
> This patch fix the bug when setting max-lipo-group in AutoFDO:
>
> Bootstrapped and passed regression test.
>
> OK for google branches?
>
> Thanks,
> Dehao
>
> Index: gcc/auto-profile.c
> ===================================================================
> --- gcc/auto-profile.c (revision 202926)
> +++ gcc/auto-profile.c (working copy)
> @@ -746,7 +746,7 @@ read_aux_modules (void)
>      "assembler statements", *iter);
>    continue;
>   }
> -      if (max_group != 0 && curr_module == max_group)
> +      if (max_group != 0 && curr_module >= max_group)
>   {
>    if (flag_opt_info)
>      inform (0, "Not importing %s: maximum group size reached", *iter);

Patch

Index: gcc/auto-profile.c
===================================================================
--- gcc/auto-profile.c (revision 202926)
+++ gcc/auto-profile.c (working copy)
@@ -746,7 +746,7 @@  read_aux_modules (void)
     "assembler statements", *iter);
   continue;
  }
-      if (max_group != 0 && curr_module == max_group)
+      if (max_group != 0 && curr_module >= max_group)
  {
   if (flag_opt_info)
     inform (0, "Not importing %s: maximum group size reached", *iter);