This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

[patch] Remove redundant intrinsics

From: "Koval, Julia" <julia dot koval at intel dot com>
To: 'GCC Patches' <gcc-patches at gcc dot gnu dot org>
Cc: Kirill Yukhin <kirill dot yukhin at gmail dot com>
Date: Mon, 4 Jun 2018 10:27:32 +0000
Subject: [patch] Remove redundant intrinsics
Dlp-product: dlpe-windows
Dlp-reaction: no-action
Dlp-version: 11.0.200.100

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.

Thanks,
Julia

Attachment: remove_in.patch
Description: remove_in.patch

Follow-Ups:
- Re: [patch] Remove redundant intrinsics
  - From: Kirill Yukhin
- Re: [patch] Remove redundant intrinsics
  - From: H.J. Lu

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]