>>> Here's the new patch. The only changes from the earlier patch are in >>> handle_missing_profiles, where we now get the counts off of the entry >>> and call stmt bbs, and in tree_profiling, where we call >>> handle_missing_profiles earlier and I have removed the outlined cgraph >>> rebuilding code since it doesn't need to be reinvoked. >>> >>> Honza, does this look ok for trunk when stage 1 reopens? David, I can >>> send a similar patch for review to google-4_8 if it looks good. >>> >>> Thanks, >>> Teresa >>> ... > > Spec testing of my earlier patch hit an issue with the call to > gimple_bb in this routine, since the caller was a thunk and therefore > the edge did not have a call_stmt set. I've attached a slightly > modified patch that guards the call by a check to > cgraph_function_with_gimple_body_p. Regression and spec testing are > clean. > > Teresa
Ping - Honza, does this patch look ok for stage 1? Thanks, Teresa -- Teresa Johnson | Software Engineer | tejohn...@google.com | 408-460-2413