On Thu, Jan 26, 2017 at 04:48:07AM -0800, Kirill Yukhin wrote:
> Hello,
> Two tests are failing for me on 32b variant bacause of wrong constants.
> Fixed. Tests now pass (under SDE).
>
> gcc/testsuite/
> * gcc.target/i386/avx512bw-kshiftlq-2.c: Use unsigned long long
> constant.
> * gcc.target/i386/avx512bw-kshiftrq-2.c: Ditto.
>
> If no objections from RMs I'll check it into main trunk in couple of hrs.
>
> After this fix whole AVX-512 testsuite pass for me under SDE.
This is ok.
> diff --git a/gcc/testsuite/gcc.target/i386/avx512bw-kshiftlq-2.c
> b/gcc/testsuite/gcc.target/i386/avx512bw-kshiftlq-2.c
> index 4dabb4a..f578625 100644
> --- a/gcc/testsuite/gcc.target/i386/avx512bw-kshiftlq-2.c
> +++ b/gcc/testsuite/gcc.target/i386/avx512bw-kshiftlq-2.c
> @@ -11,6 +11,6 @@ avx512bw_test ()
> unsigned int i = 53;
>
> volatile __mmask64 r = _kshiftli_mask64 (k1, i);
> - if (r != 1 << i)
> + if (r != 1ULL << i)
> abort ();
> }
> diff --git a/gcc/testsuite/gcc.target/i386/avx512bw-kshiftrq-2.c
> b/gcc/testsuite/gcc.target/i386/avx512bw-kshiftrq-2.c
> index 655f926..c6ba0b2 100644
> --- a/gcc/testsuite/gcc.target/i386/avx512bw-kshiftrq-2.c
> +++ b/gcc/testsuite/gcc.target/i386/avx512bw-kshiftrq-2.c
> @@ -1,3 +1,4 @@
> +
> /* { dg-do run } */
> /* { dg-options "-O2 -mavx512bw" } */
> /* { dg-require-effective-target avx512bw } */
> @@ -8,7 +9,7 @@ void
> avx512bw_test ()
> {
> unsigned int i = 53;
> - __mmask64 k1 = 1 << i;
> + __mmask64 k1 = 1ULL << i;
>
> volatile __mmask64 r = _kshiftri_mask64 (k1, i);
> if (r != 1)
Jakub
