Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-22 Thread Michael Matz
Hi, On Mon, 21 Nov 2016, Richard Biener wrote: > >> Btw, this probably means that on haswell (or other archs with vf==4) mgrid > >> is slower than necessary. On mgrid you really really want predictive > >> commoning to happen. Vectorization isn't that interesting there. > > Interesting, I will

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-21 Thread Richard Biener
On Fri, Nov 18, 2016 at 5:57 PM, Bin.Cheng wrote: > On Fri, Nov 18, 2016 at 4:52 PM, Michael Matz wrote: >> Hi, >> >> On Thu, 17 Nov 2016, Bin.Cheng wrote: >> >>> B) Depending on ilp, I think below test strings fail for long time with >>> haswell: >>> ! {

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-18 Thread Bin.Cheng
On Fri, Nov 18, 2016 at 4:52 PM, Michael Matz wrote: > Hi, > > On Thu, 17 Nov 2016, Bin.Cheng wrote: > >> B) Depending on ilp, I think below test strings fail for long time with >> haswell: >> ! { dg-final { scan-tree-dump-times "Executing predictive commoning >> without unrolling"

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-18 Thread Michael Matz
Hi, On Thu, 17 Nov 2016, Bin.Cheng wrote: > B) Depending on ilp, I think below test strings fail for long time with > haswell: > ! { dg-final { scan-tree-dump-times "Executing predictive commoning > without unrolling" 1 "pcom" { target lp64 } } } > ! { dg-final { scan-tree-dump-times "Executing

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-17 Thread Bin.Cheng
On Thu, Nov 17, 2016 at 10:53 AM, Richard Biener wrote: > On Thu, Nov 17, 2016 at 11:26 AM, Bin.Cheng wrote: >> On Thu, Nov 17, 2016 at 8:32 AM, Richard Biener >> wrote: >>> On Wed, Nov 16, 2016 at 6:20 PM, Bin Cheng

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-17 Thread Richard Biener
On Thu, Nov 17, 2016 at 11:26 AM, Bin.Cheng wrote: > On Thu, Nov 17, 2016 at 8:32 AM, Richard Biener > wrote: >> On Wed, Nov 16, 2016 at 6:20 PM, Bin Cheng wrote: >>> Hi, >>> Currently test

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-17 Thread Bin.Cheng
On Thu, Nov 17, 2016 at 8:32 AM, Richard Biener wrote: > On Wed, Nov 16, 2016 at 6:20 PM, Bin Cheng wrote: >> Hi, >> Currently test gfortran.dg/vect/fast-math-mgrid-resid.f checks all >> predictive commoning opportunities for all possible loops.

Re: [PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-17 Thread Richard Biener
On Wed, Nov 16, 2016 at 6:20 PM, Bin Cheng wrote: > Hi, > Currently test gfortran.dg/vect/fast-math-mgrid-resid.f checks all predictive > commoning opportunities for all possible loops. This makes it fragile > because vectorizer may peel the loop differently, as well as may

[PATCH PR78114]Refine gfortran.dg/vect/fast-math-mgrid-resid.f

2016-11-16 Thread Bin Cheng
Hi, Currently test gfortran.dg/vect/fast-math-mgrid-resid.f checks all predictive commoning opportunities for all possible loops. This makes it fragile because vectorizer may peel the loop differently, as well as may choose different vector factors. For example, on x86-solaris, vectorizer