RE: [PATCH v2] i386: Fix AVX512 intrin macro typo

2024-07-29 Thread Jiang, Haochen
> -Original Message- > From: Jakub Jelinek > Sent: Monday, July 29, 2024 4:41 PM > To: Jiang, Haochen > Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ; > ubiz...@gmail.com > Subject: Re: [PATCH v2] i386: Fix AVX512 intrin macro typo > > On Mon, Jul 29, 2024

Re: [PATCH v2] i386: Fix AVX512 intrin macro typo

2024-07-29 Thread Jakub Jelinek
On Mon, Jul 29, 2024 at 02:07:24AM +, Jiang, Haochen wrote: > > LGTM with the above ChangeLog nit fixed, for trunk/release branches, even > > for > > 14.2 if committed RSN. > > Ok. I will commit them and backport them to GCC13 and GCC12 now. For GCC14, > we could wait for GCC14.3 since it has

RE: [PATCH v2] i386: Fix AVX512 intrin macro typo

2024-07-28 Thread Jiang, Haochen
> -Original Message- > From: Jakub Jelinek > Sent: Friday, July 26, 2024 7:59 PM > To: Jiang, Haochen > Cc: gcc-patches@gcc.gnu.org; Liu, Hongtao ; > ubiz...@gmail.com > Subject: Re: [PATCH v2] i386: Fix AVX512 intrin macro typo > > On Fri, Jul 26, 2024 a

Re: [PATCH v2] i386: Fix AVX512 intrin macro typo

2024-07-26 Thread Jakub Jelinek
On Fri, Jul 26, 2024 at 04:10:48PM +0800, Haochen Jiang wrote: > * config/i386/avx512dqintrin.h > (_mm_mask_fpclass_ss_mask): Correct operand order. > (_mm_mask_fpclass_sd_mask): Ditto. > (_mm_reduce_round_sd): Use -1 as mask since it is non-mask. > (_mm_reduce_round_s

Re: [PATCH v2] i386: Fix AVX512 intrin macro typo

2024-07-26 Thread Richard Biener
On Fri, Jul 26, 2024 at 10:14 AM Haochen Jiang wrote: > > Hi all, > > I have added related testcases into the patch. > > Ok for trunk and backport to GCC 14, GCC 13 and GCC 12? Hmm, it might be OK for 14.2 still, even without a new RC. But please wait until after 14.2 is released unless Jakub al

[PATCH v2] i386: Fix AVX512 intrin macro typo

2024-07-26 Thread Haochen Jiang
Hi all, I have added related testcases into the patch. Ok for trunk and backport to GCC 14, GCC 13 and GCC 12? Thx, Haochen --- Changes in v2: Add related testcases --- There are several typo in AVX512 intrins macro define. Correct them to solve errors when compiled with -O0. gcc/ChangeLog