RE: [PATCH] i386: Share AES xmm intrin with VAES

2023-04-18 Thread Liu, Hongtao via Gcc-patches
> -Original Message- > From: Jiang, Haochen > Sent: Wednesday, April 19, 2023 10:41 AM > To: Hongtao Liu > Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ; > ubiz...@gmail.com > Subject: RE: [PATCH] i386: Share AES xmm intrin with VAES > > > > a/gcc/con

RE: [PATCH] i386: Share AES xmm intrin with VAES

2023-04-18 Thread Jiang, Haochen via Gcc-patches
> > a/gcc/config/i386/sse.md b/gcc/config/i386/sse.md index > > 33e281901cf..e7d565a8389 100644 > > --- a/gcc/config/i386/sse.md > > +++ b/gcc/config/i386/sse.md > > @@ -25107,67 +25107,71 @@ > > > > ;; > > ;; > > > > (define_insn

Re: [PATCH] i386: Share AES xmm intrin with VAES

2023-04-18 Thread Hongtao Liu via Gcc-patches
On Tue, Apr 18, 2023 at 3:19 PM Haochen Jiang via Gcc-patches wrote: > > Hi all, > > Currently in GCC, the 128 bit intrin for instruction vaes{end,dec}{last,} > is under AES ISA. Because there is no dependency between ISA set AES > and VAES, The 128 bit intrin is not available when we use compiler

[PATCH] i386: Share AES xmm intrin with VAES

2023-04-18 Thread Haochen Jiang via Gcc-patches
Hi all, I realized that I attached a old version of my patch. We should change the error message of pr109117-1.c but not pr84335.c. Please review this patch. Thx, Haochen gcc/ChangeLog: * common/config/i386/i386-common.cc (OPTION_MASK_ISA2_AVX_UNSET): Add OPTION_MASK_ISA2_VAES_

[PATCH] i386: Share AES xmm intrin with VAES

2023-04-18 Thread Haochen Jiang via Gcc-patches
Hi all, Currently in GCC, the 128 bit intrin for instruction vaes{end,dec}{last,} is under AES ISA. Because there is no dependency between ISA set AES and VAES, The 128 bit intrin is not available when we use compiler flag -mvaes -mavx512vl and there is no other way to use that intrin. But it shou