diff mbox

[GOOGLE] Updates SSA after VPT transofrmations in AFDO pass

Message ID CAO2gOZWK_yiiFO7u+Bhtrf6rJhCfwQj8mgD2zBYypqsM4nA_DA@mail.gmail.com
State New
Headers show

Commit Message

Dehao Chen April 3, 2014, 5:56 p.m. UTC 
This patch updates SSA after VPT transformation. This is needed
because compute_inline_parameters will ICE without updated SSA.

Testing on-going.

OK for google-4_8?

Thanks,
Dehao

Comments

Xinliang David Li April 3, 2014, 6:02 p.m. UTC | #1 
looks fine.

David

On Thu, Apr 3, 2014 at 10:56 AM, Dehao Chen <dehao@google.com> wrote:
> This patch updates SSA after VPT transformation. This is needed
> because compute_inline_parameters will ICE without updated SSA.
>
> Testing on-going.
>
> OK for google-4_8?
>
> Thanks,
> Dehao
>
> Index: gcc/auto-profile.c
> ===================================================================
> --- gcc/auto-profile.c (revision 209059)
> +++ gcc/auto-profile.c (working copy)
> @@ -1448,6 +1448,7 @@ afdo_vpt_for_early_inline (stmt_set *promoted_stmt
>        free_dominance_info (CDI_POST_DOMINATORS);
>        calculate_dominance_info (CDI_POST_DOMINATORS);
>        calculate_dominance_info (CDI_DOMINATORS);
> +      update_ssa (TODO_update_ssa);
>        rebuild_cgraph_edges ();
>        return true;
>      }
diff mbox

Patch

Index: gcc/auto-profile.c
===================================================================
--- gcc/auto-profile.c (revision 209059)
+++ gcc/auto-profile.c (working copy)
@@ -1448,6 +1448,7 @@  afdo_vpt_for_early_inline (stmt_set *promoted_stmt
       free_dominance_info (CDI_POST_DOMINATORS);
       calculate_dominance_info (CDI_POST_DOMINATORS);
       calculate_dominance_info (CDI_DOMINATORS);
+      update_ssa (TODO_update_ssa);
       rebuild_cgraph_edges ();
       return true;
     }