[PATCH, rs6000] Update default loop peel limits

[Michael Matz]

Hi,

On Fri, 7 Dec 2012, Jan Hubicka wrote:

> > > Actually the calculix regression is also seen on core.
> > > Igor was looking into what loops got slower and why. Either we can fix that partiuclar
> > > loop or revert to the old default (that sadly causes quite a lot of code bloat)
> > 
> > Well, as the patch regressed the testcase in put in the testsuite for calculix
> > that is no surprise... gfortran.dg/reassoc_4.f.  The fix for the testcase was to
> > increase the limit with a --param :/
> 
> I would not care too much about limits for gfortran.dg/reassoc_4.f, it 
> is an artificail testcase.

Ehm, no it's not.  It _specifically_ is the important inner loop nest of 
calculix and it should be compiled to exactly the expected form (i.e. 22 
multiplications and unrolled) without any changes to params.  At least 
when you expect that -Ofast is giving good results on 454.calculix.

I.e. this testcase breaking automatically implies slower calculix results.


Ciao,
Michael.