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?
> 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 } */
> }
Jakub
