Good for google branches. Need to measure the performance and size impact on SPEC2k/06 with this patch and the previous hotcaller patch for trunk.
thanks, David On Wed, Aug 24, 2011 at 2:43 PM, Mark Heffernan <meh...@google.com> wrote: > This patch bumps up the parameter 'hot-bb-count-fraction' from 10000 > to 40000. This results in about a 0.5% geomean performance > improvement across internal benchmarks for x86-64 LIPO. The parameter > change effectively increases the number of functions/callsites which > are considered hot. The performance improvement is likely due to > increased inlining (more callsites are considered hot and available > for inlining). > > Bootstrapped and reg-tested on x86-64. OK for google/gcc-4_6? > > Mark > > > 2011-08-24 Mark Heffernan <meh...@google.com> > > * params.def (hot-bb-count-fraction): Change default value. > > > > Index: params.def > =================================================================== > --- params.def (revision 177964) > +++ params.def (working copy) > @@ -382,7 +382,7 @@ DEFPARAM(PARAM_SMS_LOOP_AVERAGE_COUNT_TH > DEFPARAM(HOT_BB_COUNT_FRACTION, > "hot-bb-count-fraction", > "Select fraction of the maximal count of repetitions of basic > block in program given basic block needs to have to be considered > hot", > - 10000, 0, 0) > + 40000, 0, 0) > DEFPARAM(HOT_BB_FREQUENCY_FRACTION, > "hot-bb-frequency-fraction", > "Select fraction of the maximal frequency of executions of > basic block in function given basic block needs to have to be > considered hot", >
