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: >>> ! { dg-final { scan-tree-dump-times "Executi

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" 1 "pcom" { tar

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 wrote: Hi, Currently test gfortran.dg/vect/fast-math-mgrid-resid.f c

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 gfortran.dg/vect/fast-math-mgrid-resid.f checks all >>> predictive commoning opportunities for all possib

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. This makes it >> fragile because vectorizer may

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 choose > different