On Sat, Mar 21, 2015 at 01:47:10AM -0600, Jeff Law wrote:
> On 03/16/2015 01:27 PM, Jakub Jelinek wrote:
> >
> >What effect does the patch have on compile time on say x86_64 or ppc64?
> I bootstrapped x86_64 trunk then timed compiling 640 .i/.ii files with -O2.
> That was repeated 10 times (to get a sense of variability).  Each run took a
> little over 40 minutes with variability of less than a second (slowest run
> compared to fastest run).
> 
> I then also gathered data for a smaller set of runs for -O2 -funroll-loops
> (given the consistency between runs, 10 iterations seemed like major
> overkill).  Again variability was less than a second.
> 
> I then applied the patch and repeated the -O2 and -O2 -funroll-loops test.
> The patched compiler was within a second of the unpatched compiler for both
> the -O2 and -O2 -funroll-loops tests.  Given the difference was within the
> noise level of those tests, my conclusion is the new code makes no
> measurable difference in compile time performance for x86_64.
> 
> Similar tests are in progress on powerpc64-linux-gnu.

Thanks for the testing.  I think we want the patch in now.

        Jakub

Reply via email to