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 *

Vectorize stores with unknown stride

2015-05-06 Thread Michael Matz
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 * __restrict src, float * __restrict src2, int stride, int n