Re: Vectorize stores with unknown stride

2015-05-07 Thread Alan Lawrence
NP, and sorry for the spurious comments, hadn't spotted you were using nunits. I like the testcase, thanks :). A. Michael Matz wrote: On Thu, 7 May 2015, Alan Lawrence wrote: Also update comment? (5 identical cases) Also update comment? Obviously a good idea, thanks :) (s/loads/accesses/

Re: Vectorize stores with unknown stride

2015-05-07 Thread Michael Matz
On Thu, 7 May 2015, Alan Lawrence wrote: > Also update comment? (5 identical cases) > > Also update comment? Obviously a good idea, thanks :) (s/loads/accesses/ for the commit) > > @@ -5013,7 +5025,7 @@ vectorizable_store (gimple stmt, gimple_stmt_iterator > > *gsi, gimple *vec_stmt, > >tr

Re: Vectorize stores with unknown stride

2015-05-07 Thread Alan Lawrence
(Below are all minor/style points only, no reason for patch not to go in.) Michael Matz wrote: diff --git a/gcc/tree-vect-data-refs.c b/gcc/tree-vect-data-refs.c index 96afc7a..6d8f17e 100644 --- a/gcc/tree-vect-data-refs.c +++ b/gcc/tree-vect-data-refs.c @@ -665,7 +665,7 @@ vect_compute_data_re

Re: Vectorize stores with unknown stride

2015-05-07 Thread Richard Biener
On Wed, May 6, 2015 at 5:37 PM, Michael Matz wrote: > Hi, > > I'm sitting on this since quite some time already and always missed stage > 1. This implements support for vectorizing strided stores with unknown > but loop invariant stride, like: > > sumit (float * __restrict dest, >float *