Re: [PATCH][i386,AVX] Enable VBMI2 support [1/7]

2017-11-15 Thread Kirill Yukhin 
Hello Julia!
On 25 Oct 11:18, Koval, Julia wrote:
> Thanks, fix it.
> 
> gcc/
>   * common/config/i386/i386-common.c (OPTION_MASK_ISA_AVX512VBMI2_SET,
>   OPTION_MASK_ISA_AVX512VBMI2_UNSET): New.
>   (ix86_handle_option): Handle -mavx512vbmi2.
>   * config/i386/cpuid.h: Add bit_AVX512VBMI2.
>   * config/i386/driver-i386.c (host_detect_local_cpu): Handle new bit.
>   * config/i386/i386-c.c (__AVX512VBMI2__): New.
>   * config/i386/i386.c (ix86_target_string): Handle -mavx512vbmi2.
>   (ix86_valid_target_attribute_inner_p): Ditto.
>   * config/i386/i386.h (TARGET_AVX512VBMI2, TARGET_AVX512VBMI2_P): New.
>   * config/i386/i386.opt (mavx512vbmi2): New option.
>   * doc/invoke.texi: Add new option.
Your patch is OK. I've checked it into main trunk.

--
Thanks, K

RE: [PATCH][i386,AVX] Enable VBMI2 support [1/7]

2017-10-25 Thread Koval, Julia 
Thanks, fix it.

gcc/
* common/config/i386/i386-common.c (OPTION_MASK_ISA_AVX512VBMI2_SET,
OPTION_MASK_ISA_AVX512VBMI2_UNSET): New.
(ix86_handle_option): Handle -mavx512vbmi2.
* config/i386/cpuid.h: Add bit_AVX512VBMI2.
* config/i386/driver-i386.c (host_detect_local_cpu): Handle new bit.
* config/i386/i386-c.c (__AVX512VBMI2__): New.
* config/i386/i386.c (ix86_target_string): Handle -mavx512vbmi2.
(ix86_valid_target_attribute_inner_p): Ditto.
* config/i386/i386.h (TARGET_AVX512VBMI2, TARGET_AVX512VBMI2_P): New.
* config/i386/i386.opt (mavx512vbmi2): New option.
* doc/invoke.texi: Add new option.


> -Original Message-
> From: Joseph Myers [mailto:jos...@codesourcery.com]
> Sent: Wednesday, October 25, 2017 12:40 AM
> To: Koval, Julia <julia.ko...@intel.com>
> Cc: GCC Patches <gcc-patches@gcc.gnu.org>; Kirill Yukhin
> <kirill.yuk...@gmail.com>
> Subject: Re: [PATCH][i386,AVX] Enable VBMI2 support [1/7]
> 
> On Tue, 24 Oct 2017, Koval, Julia wrote:
> 
> > config/i386/i386.opt (mavx512vbmi2): New option.
> 
> Any patch adding a new command-line option needs to add documentation of
> it to invoke.texi.
> 
> --
> Joseph S. Myers
> jos...@codesourcery.com


0001-VBMI2-option.patch
Description: 0001-VBMI2-option.patch

Re: [PATCH][i386,AVX] Enable VBMI2 support [1/7]

2017-10-24 Thread Joseph Myers 
On Tue, 24 Oct 2017, Koval, Julia wrote:

>   config/i386/i386.opt (mavx512vbmi2): New option.

Any patch adding a new command-line option needs to add documentation of 
it to invoke.texi.

-- 
Joseph S. Myers
jos...@codesourcery.com

[PATCH][i386,AVX] Enable VBMI2 support [1/7]

2017-10-24 Thread Koval, Julia 
Hi,
This patch enables VBMI2 isaset option. The doc for isaset and instruction: 
https://software.intel.com/sites/default/files/managed/c5/15/architecture-instruction-set-extensions-programming-reference.pdf

Ok for trunk?

Thanks,
Julia

gcc/
common/config/i386/i386-common.c (OPTION_MASK_ISA_AVX512VBMI2_SET,
OPTION_MASK_ISA_AVX512VBMI2_UNSET): New.
(ix86_handle_option): Handle -mavx512vbmi2.
config/i386/cpuid.h: Add bit_AVX512VBMI2.
config/i386/driver-i386.c (host_detect_local_cpu): Handle new bit.
config/i386/i386-c.c (__AVX512VBMI2__): New.
config/i386/i386.c (ix86_target_string): Handle -mavx512vbmi2.
(ix86_valid_target_attribute_inner_p): Ditto.
config/i386/i386.h (TARGET_AVX512VBMI2, TARGET_AVX512VBMI2_P): New.
config/i386/i386.opt (mavx512vbmi2): New option.


0003-VBMI2-option.patch
Description: 0003-VBMI2-option.patch