[patch] Remove redundant intrinsics

Kirill Yukhin kirill.yukhin@gmail.com
Mon Jun 11 11:18:00 GMT 2018


Hello Julia!
On 04 июн 10:27, Koval, Julia wrote:
> Hi,
> 
> Since pre-Icelake ISA already had 128bit version vpclmul and vaes, we already have intrinsics for them(_mm_aesdec_si128, _mm_aesdeclast_si128, _mm_aesenc_si128, _mm_aesenclast_si128, _mm_clmulepi64_si128). Therefore intrinsics for them, introduced with Icelake instructions are redundant. This patch removes them. Ok for trunk?
> 
> gcc/
> 	* config/i386/vaesintrin.h (_mm_aesdec_epi128, _mm_aesdeclast_epi128,
> 	_mm_aesenc_epi128, _mm_aesenclast_epi128): Remove.
> 	* config/i386/vpclmulqdqintrin.h (_mm_clmulepi64_epi128): Remove.
> 
> gcc/testsuite/
> 	* gcc.target/i386/avx512fvl-vaes-1.c: Remove 128bit versions from test.
> 	* gcc.target/i386/vpclmulqdq.c: Ditto.
Your patch is OK. I've checked it into main trunk.

> 
> Thanks,
> Julia

--
Thanks, K



More information about the Gcc-patches mailing list