Re: [PATCH, obvious, AVX-512] Add missing AVX-512 features detection.

[Kirill Yukhin]

Hi Richard,
On 06 Oct 09:36, Richard Biener wrote:
> The test now execute FAILs for me:
> 
> FAIL: gcc.target/i386/builtin_target.c execution test
> 
> I have family 6, model 94
Wow, Skylake!

Fixed. AVX-512VBMI bit lives in ecx, not ebx as rest of AVX-512.

gcc/testsuite/
        * gcc.target/i386/builtin_target.c: Fix AVX-512VBMI detection.

I've checked the test on Skylake machine and it passes now.

--
Thanks, K

Index: gcc/testsuite/gcc.target/i386/builtin_target.c
===================================================================
--- gcc/testsuite/gcc.target/i386/builtin_target.c      (revision 228513)
+++ gcc/testsuite/gcc.target/i386/builtin_target.c      (working copy)
@@ -211,7 +211,7 @@
        assert (__builtin_cpu_supports ("avx512dq"));
       if (ebx & bit_AVX512IFMA)
        assert (__builtin_cpu_supports ("avx512ifma"));
-      if (ebx & bit_AVX512VBMI)
+      if (ecx & bit_AVX512VBMI)
        assert (__builtin_cpu_supports ("avx512vbmi"));
     }
 }