ping... Thanks, Dehao
On Tue, Nov 18, 2014 at 2:29 PM, Dehao Chen <de...@google.com> wrote: > This patch updates ssa and inline summary in the correct location for AutoFDO. > > Bootstrapped and passed regression test. OK for trunk? > > Thanks, > Dehao > > gcc/ChangeLog: > 2014-11-18 Dehao Chen <de...@google.com> > > * auto-profile.c (afdo_annotate_cfg): Invoke update_ssa in the right > place. > (auto_profile): Recompute inline summary after processing cgraph node. > > Index: gcc/auto-profile.c > =================================================================== > --- gcc/auto-profile.c (revision 217741) > +++ gcc/auto-profile.c (working copy) > @@ -1514,7 +1514,14 @@ afdo_annotate_cfg (const stmt_set &promoted_stmts) > profile_status_for_fn (cfun) = PROFILE_READ; > } > if (flag_value_profile_transformations) > - gimple_value_profile_transformations (); > + { > + gimple_value_profile_transformations (); > + free_dominance_info (CDI_DOMINATORS); > + free_dominance_info (CDI_POST_DOMINATORS); > + calculate_dominance_info (CDI_POST_DOMINATORS); > + calculate_dominance_info (CDI_DOMINATORS); > + update_ssa (TODO_update_ssa); > + } > } > > /* Wrapper function to invoke early inliner. */ > @@ -1592,7 +1599,6 @@ auto_profile (void) > early_inline (); > autofdo::afdo_annotate_cfg (promoted_stmts); > compute_function_frequency (); > - update_ssa (TODO_update_ssa); > > /* Local pure-const may imply need to fixup the cfg. */ > if (execute_fixup_cfg () & TODO_cleanup_cfg) > @@ -1601,6 +1607,7 @@ auto_profile (void) > free_dominance_info (CDI_DOMINATORS); > free_dominance_info (CDI_POST_DOMINATORS); > cgraph_edge::rebuild_edges (); > + compute_inline_parameters (cgraph_node::get (current_function_decl), > true); > pop_cfun (); > }