Re: [GOOGLE] Remove zero_edge propagation algorithm

2013-11-22 Thread Xinliang David Li
ok thanks. David On Fri, Nov 22, 2013 at 1:59 PM, Dehao Chen wrote: > Patch updated... > > There is no performance change with/without the patch. I think this > was used to workaround the debug info accuracy issue. But after debug > info is more improved now, the heuristic is not needed any more

Re: [GOOGLE] Remove zero_edge propagation algorithm

2013-11-22 Thread Dehao Chen
Patch updated... There is no performance change with/without the patch. I think this was used to workaround the debug info accuracy issue. But after debug info is more improved now, the heuristic is not needed any more. Thanks, Dehao Index: gcc/auto-profile.c

Re: [GOOGLE] Remove zero_edge propagation algorithm

2013-11-22 Thread Xinliang David Li
On Fri, Nov 22, 2013 at 12:27 PM, Dehao Chen wrote: > This patch removes the zero_edge heuristic during profile propagation. > The zero_edge heuristic does not seem to be effective in improving > performance. "not effective" here means degrading performance? David > > Tested: > Bootstrapped and

Re: [GOOGLE] Remove zero_edge propagation algorithm

2013-11-22 Thread Diego Novillo
On Fri, Nov 22, 2013 at 3:27 PM, Dehao Chen wrote: > This patch removes the zero_edge heuristic during profile propagation. > The zero_edge heuristic does not seem to be effective in improving > performance. > > Tested: > Bootstrapped and passed regression test and performance test. > > OK for goo

[GOOGLE] Remove zero_edge propagation algorithm

2013-11-22 Thread Dehao Chen
This patch removes the zero_edge heuristic during profile propagation. The zero_edge heuristic does not seem to be effective in improving performance. Tested: Bootstrapped and passed regression test and performance test. OK for google-4_8? Thanks, Dehao Index: gcc/auto-profile.c ===