Re: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake isa

2018-03-14 Thread Kirill Yukhin
Hello Julia!



> On 14 Mar 2018, at 10:48, Koval, Julia <julia.ko...@intel.com> wrote:
> 
> Gentle ping.
Your patch is OK for main trunk.

—
Thanks, K


>> -Original Message-
>> From: Koval, Julia
>> Sent: Monday, February 12, 2018 10:57 AM
>> To: Kirill Yukhin <kirill.yuk...@gmail.com>
>> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>
>> Subject: RE: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake 
>> isa
>> 
>> Hi,
>> 
>> There is no PR for this. This builtin was just missing for all new cpus.
>> 
>> Thanks,
>> Julia
>> 
>>> -Original Message-
>>> From: Kirill Yukhin [mailto:kirill.yuk...@gmail.com]
>>> Sent: Monday, February 12, 2018 7:19 AM
>>> To: Koval, Julia <julia.ko...@intel.com>
>>> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>
>>> Subject: Re: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake
>> isa
>>> 
>>> Hello Julia.
>>> 
>>> On 15 Jan 08:28, Koval, Julia wrote:
>>>> Hi,
>>>> This patch fixes subj. Ok for trunk?
>>>> 
>>>> gcc/
>>>>* config/i386/i386.c (F_AVX512VBMI2, F_GFNI, F_VPCLMULQDQ,
>>> F_AVX512VNNI,
>>>>F_AVX512BITALG): New.
>>>> 
>>>> gcc/testsuite/
>>>>* gcc.target/i386/builtin_target.c (check_intel_cpu_model): Add
>>> cannonlake.
>>>>(check_features): Add avx512vbmi2, gfni, vpclmulqdq, avx512vnni,
>>>>avx512bitalg.
>>>> 
>>>> libgcc/
>>>>* config/i386/cpuinfo.c (get_available_features): Add
>>> FEATURE_AVX512VBMI2,
>>>>FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI,
>>> FEATURE_AVX512BITALG.
>>>>* config/i386/cpuinfo.h (processor_features) Add
>>> FEATURE_AVX512VBMI2,
>>>>FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI,
>>> FEATURE_AVX512BITALG.
>>> 
>>> Could you pls mention, which problem does your patch fix?
>>> 
>>> --
>>> Thanks, K



RE: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake isa

2018-03-14 Thread Koval, Julia
Gentle ping.

> -Original Message-
> From: Koval, Julia
> Sent: Monday, February 12, 2018 10:57 AM
> To: Kirill Yukhin <kirill.yuk...@gmail.com>
> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>
> Subject: RE: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake 
> isa
> 
> Hi,
> 
> There is no PR for this. This builtin was just missing for all new cpus.
> 
> Thanks,
> Julia
> 
> > -Original Message-
> > From: Kirill Yukhin [mailto:kirill.yuk...@gmail.com]
> > Sent: Monday, February 12, 2018 7:19 AM
> > To: Koval, Julia <julia.ko...@intel.com>
> > Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>
> > Subject: Re: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake
> isa
> >
> > Hello Julia.
> >
> > On 15 Jan 08:28, Koval, Julia wrote:
> > > Hi,
> > > This patch fixes subj. Ok for trunk?
> > >
> > > gcc/
> > >   * config/i386/i386.c (F_AVX512VBMI2, F_GFNI, F_VPCLMULQDQ,
> > F_AVX512VNNI,
> > >   F_AVX512BITALG): New.
> > >
> > > gcc/testsuite/
> > >   * gcc.target/i386/builtin_target.c (check_intel_cpu_model): Add
> > cannonlake.
> > >   (check_features): Add avx512vbmi2, gfni, vpclmulqdq, avx512vnni,
> > >   avx512bitalg.
> > >
> > > libgcc/
> > >   * config/i386/cpuinfo.c (get_available_features): Add
> > FEATURE_AVX512VBMI2,
> > >   FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI,
> > FEATURE_AVX512BITALG.
> > >   * config/i386/cpuinfo.h (processor_features) Add
> > FEATURE_AVX512VBMI2,
> > >   FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI,
> > FEATURE_AVX512BITALG.
> >
> > Could you pls mention, which problem does your patch fix?
> >
> > --
> > Thanks, K


RE: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake isa

2018-02-12 Thread Koval, Julia
Hi,

There is no PR for this. This builtin was just missing for all new cpus.

Thanks,
Julia

> -Original Message-
> From: Kirill Yukhin [mailto:kirill.yuk...@gmail.com]
> Sent: Monday, February 12, 2018 7:19 AM
> To: Koval, Julia <julia.ko...@intel.com>
> Cc: 'GCC Patches' <gcc-patches@gcc.gnu.org>
> Subject: Re: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake 
> isa
> 
> Hello Julia.
> 
> On 15 Jan 08:28, Koval, Julia wrote:
> > Hi,
> > This patch fixes subj. Ok for trunk?
> >
> > gcc/
> > * config/i386/i386.c (F_AVX512VBMI2, F_GFNI, F_VPCLMULQDQ,
> F_AVX512VNNI,
> > F_AVX512BITALG): New.
> >
> > gcc/testsuite/
> > * gcc.target/i386/builtin_target.c (check_intel_cpu_model): Add
> cannonlake.
> > (check_features): Add avx512vbmi2, gfni, vpclmulqdq, avx512vnni,
> > avx512bitalg.
> >
> > libgcc/
> > * config/i386/cpuinfo.c (get_available_features): Add
> FEATURE_AVX512VBMI2,
> > FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI,
> FEATURE_AVX512BITALG.
> > * config/i386/cpuinfo.h (processor_features) Add
> FEATURE_AVX512VBMI2,
> > FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI,
> FEATURE_AVX512BITALG.
> 
> Could you pls mention, which problem does your patch fix?
> 
> --
> Thanks, K


Re: [x86,avx] Fix __builtin_cpu_supports for icelake and cannonlake isa

2018-02-11 Thread Kirill Yukhin
Hello Julia.

On 15 Jan 08:28, Koval, Julia wrote:
> Hi,
> This patch fixes subj. Ok for trunk?
> 
> gcc/
>   * config/i386/i386.c (F_AVX512VBMI2, F_GFNI, F_VPCLMULQDQ, F_AVX512VNNI,
>   F_AVX512BITALG): New.
> 
> gcc/testsuite/
>   * gcc.target/i386/builtin_target.c (check_intel_cpu_model): Add 
> cannonlake.
>   (check_features): Add avx512vbmi2, gfni, vpclmulqdq, avx512vnni,
>   avx512bitalg.
> 
> libgcc/
>   * config/i386/cpuinfo.c (get_available_features): Add 
> FEATURE_AVX512VBMI2,
>   FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI, 
> FEATURE_AVX512BITALG.
>   * config/i386/cpuinfo.h (processor_features) Add FEATURE_AVX512VBMI2,
>   FEATURE_GFNI, FEATURE_VPCLMULQDQ, FEATURE_AVX512VNNI, 
> FEATURE_AVX512BITALG.

Could you pls mention, which problem does your patch fix?

--
Thanks, K