On Wed, Nov 22, 2017 at 02:51:09PM +0100, Richard Biener wrote:
> On Wed, Nov 22, 2017 at 10:30 AM, Richard Sandiford
> <richard.sandif...@linaro.org> wrote:
> > r254589 was supposed to leave tests unchanged for the default setting
> > of VECTOR_BITS, but I must have got my sums wrong on pr81136.c.
> > Sorry for the breakage.
> >
> > Tested on aarch64-linux-gnu, x86_64-linux-gnu and powerpc64le-linux-gnu,
> > OK to install?
>
> Ok.
That will still FAIL e.g. with -march=skylake-avx512 or -march=knl
(at least when not preferring 256 or 128 bit vectors), those would need
ALIGNMENT 64.
> > 2017-11-22 Richard Sandiford <richard.sandif...@linaro.org>
> >
> > gcc/testsuite/
> > PR testsuite/83004
> > * gcc.dg/vect/pr81136.c: Restore previous alignment of 32
> > in the default case.
> >
> > Index: gcc/testsuite/gcc.dg/vect/pr81136.c
> > ===================================================================
> > --- gcc/testsuite/gcc.dg/vect/pr81136.c 2017-11-22 09:21:12.538474075 +0000
> > +++ gcc/testsuite/gcc.dg/vect/pr81136.c 2017-11-22 09:28:04.836628929 +0000
> > @@ -2,7 +2,13 @@
> >
> > #include "tree-vect.h"
> >
> > -struct __attribute__((aligned (VECTOR_BITS / 8)))
> > +#if VECTOR_BITS > 256
> > +#define ALIGNMENT (VECTOR_BITS / 8)
> > +#else
> > +#define ALIGNMENT 32
> > +#endif
> > +
> > +struct __attribute__((aligned (ALIGNMENT)))
> > {
> > char misaligner;
> > int foo[100];
Jakub
