On Tue, Jul 4, 2017 at 10:35 AM, Jakub Jelinek <ja...@redhat.com> 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?
>
>> 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.
OK.
Thanks,
Uros.
>> --- 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
