On 24 Sep 11:20, Uros Bizjak wrote:
> On Thu, Sep 24, 2015 at 11:07 AM, Kirill Yukhin <kirill.yuk...@gmail.com>
> wrote:
> > Hello Uroš,
> > I've comitted (into main trunk) patch in the bottom which
> > checks first bit of ecx (instead of ebx) to verify of
> > AVX-512VBMI presents on the system (according to SDM).
> >
> > It makes those tests `SKIPPED' on KNL.
> >
> > Is it OK to check it into gcc-5-branch?
>
> Yes. Usually, I backport my patches after a couple of day of "soaking"
> in mainline, so autotesters have a chance to pick the patch and put it
> through their tests.
>
> BR,
> Uros.
Thanks! I'll wait for the end of the WW.
CC-ing gcc-patches which I gorgot.
--
K
>
> > gcc/testsuite/
> > * gcc.target/i386/avx512vbmi-check.h (main): Fix register
> > name while checking for AVX-512VBMI presence.
> >
> > --
> > Thanks, K
> >
> > diff --git a/gcc/testsuite/gcc.target/i386/avx512vbmi-check.h
> > b/gcc/testsuite/gcc.target/i386/avx512vbmi-check.h
> > index 591ff06..97aca27 100644
> > --- a/gcc/testsuite/gcc.target/i386/avx512vbmi-check.h
> > +++ b/gcc/testsuite/gcc.target/i386/avx512vbmi-check.h
> > @@ -25,7 +25,7 @@ main ()
> >
> > __cpuid_count (7, 0, eax, ebx, ecx, edx);
> >
> > - if ((avx512f_os_support ()) && ((ebx & bit_AVX512VBMI) ==
> > bit_AVX512VBMI))
> > + if ((avx512f_os_support ()) && ((ecx & bit_AVX512VBMI) ==
> > bit_AVX512VBMI))
> > {
> > do_test ();
> > #ifdef DEBUG
> >
