diff mbox

[GOOGLE] max-lipo-group parameter for AutoFDO

Message ID CAO2gOZWShrynMHsCseqqjjJoTCXi8cNjXzBxicGU++OOV+zuDw@mail.gmail.com
State New
Headers show

Commit Message

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

Bootstrapped and passed regression test.

OK for google branches?

Thanks,
Dehao

Comments

Xinliang David Li Sept. 26, 2013, 4:03 p.m. UTC | #1 
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);
diff mbox

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);