On Tue, 4 Jul 2017, Jakub Jelinek wrote:
> Hi!
>
> On Tue, Jun 27, 2017 at 12:27:25PM +0200, Jakub Jelinek wrote:
> > Fixed thusly, ok for trunk? Perhaps we should add another testcase to check
> > similarly gatherpf builtin without the lhs, but we'd need different options.
>
> I'd like to ping this patch, ok for trunk?
Ok (I thought it was quite obvious).
Thanks,
Richard.
> > 2017-06-27 Jakub Jelinek <ja...@redhat.com>
> >
> > PR target/81175
> > * gcc.target/i386/pr69255-2.c (foo): Use the return value of the
> > gather.
> >
> > --- gcc/testsuite/gcc.target/i386/pr69255-2.c.jj 2017-05-05
> > 09:19:48.000000000 +0200
> > +++ gcc/testsuite/gcc.target/i386/pr69255-2.c 2017-06-27
> > 12:20:31.697944761 +0200
> > @@ -12,7 +12,8 @@ __attribute__ ((__vector_size__ (16))) i
> > void
> > foo (const long long *p)
> > {
> > - __builtin_ia32_gather3siv4di (a, p, b, 1, 1); /* { dg-error
> > "needs isa option -m32 -mavx512vl" } */
> > + volatile __attribute__ ((__vector_size__ (32))) long long c;
> > + c = __builtin_ia32_gather3siv4di (a, p, b, 1, 1); /* {
> > dg-error "needs isa option -m32 -mavx512vl" } */
> > /* { dg-warning "AVX vector return without AVX enabled changes the ABI"
> > "" { target *-*-* } .-1 } */
> > /* { dg-warning "AVX vector argument without AVX enabled changes the
> > ABI" "" { target *-*-* } .-2 } */
> > }
