Re: [Google] AutoFDO cleanup patch

2013-05-12 Thread Dehao Chen
I've added some more cleanup to make AutoFDO isolated as well as consistent wil 4.8. Bootstrapped and passed regression test and benchmark performance test. OK for google branches? Thanks, Dehao http://codereview.appspot.com/9250047 On Sat, May 11, 2013 at 9:30 PM, Xinliang David Li wrote: >

Re: [Google] AutoFDO cleanup patch

2013-05-11 Thread Xinliang David Li
Looks good. David On Sat, May 11, 2013 at 3:53 PM, Dehao Chen wrote: > In AutoFDO, we early-inline callsites that was inlined in profiling > runs regardless of the size limit. With this change, the existing > ipa-inline tunings for AutoFDO is unnecessary: it's fine to just use > the traditional

[Google] AutoFDO cleanup patch

2013-05-11 Thread Dehao Chen
In AutoFDO, we early-inline callsites that was inlined in profiling runs regardless of the size limit. With this change, the existing ipa-inline tunings for AutoFDO is unnecessary: it's fine to just use the traditional FDO based heuristic. This patch cleans up the original tunings and make it easie