Re: [PATCH] AVX512FP16: Support vector shuffle builtins

2021-10-14 Thread Hongtao Liu via Gcc-patches
On Fri, Oct 15, 2021 at 2:15 PM Hongyu Wang wrote: > > > ix86_expand_vec_perm is only called by (define_expand "vec_perm" > > which means target, op0 and op1 must existed, and you can drop > > if(target/op0/op1) stuff. > > Yes, dropped. > > > Those checks for NULL seems reasonable according to doc

Re: [PATCH] AVX512FP16: Support vector shuffle builtins

2021-10-14 Thread Hongyu Wang via Gcc-patches
> ix86_expand_vec_perm is only called by (define_expand "vec_perm" > which means target, op0 and op1 must existed, and you can drop > if(target/op0/op1) stuff. Yes, dropped. > Those checks for NULL seems reasonable according to documents, > op0,op1,target maybe NULL. Thanks for pointing it out, d

Re: [PATCH] AVX512FP16: Support vector shuffle builtins

2021-10-14 Thread Hongtao Liu via Gcc-patches
On Fri, Oct 15, 2021 at 1:37 PM Hongyu Wang wrote: > > > This part seems not related to vector shuffle. > Yes, have separated this part to another patch and checked-in. > > Updated patch. Ok for this one? > > Hongtao Liu via Gcc-patches 于2021年10月14日周四 下午2:33写道: > > > > On Thu, Oct 14, 2021 at 10:

Re: [PATCH] AVX512FP16: Support vector shuffle builtins

2021-10-14 Thread Hongyu Wang via Gcc-patches
> This part seems not related to vector shuffle. Yes, have separated this part to another patch and checked-in. Updated patch. Ok for this one? Hongtao Liu via Gcc-patches 于2021年10月14日周四 下午2:33写道: > > On Thu, Oct 14, 2021 at 10:39 AM Hongyu Wang via Gcc-patches > wrote: > > > > Hi, > > > > This

Re: [PATCH] AVX512FP16: Support vector shuffle builtins

2021-10-13 Thread Hongtao Liu via Gcc-patches
On Thu, Oct 14, 2021 at 10:39 AM Hongyu Wang via Gcc-patches wrote: > > Hi, > > This patch supports HFmode vector shuffle by creating HImode subreg when > expanding permutation expr. > > Bootstrapped/regtested on x86_64-pc-linux-gnu{-m32,} and sde{-m32,} > OK for master? > > gcc/ChangeLog: > >

[PATCH] AVX512FP16: Support vector shuffle builtins

2021-10-13 Thread Hongyu Wang via Gcc-patches
Hi, This patch supports HFmode vector shuffle by creating HImode subreg when expanding permutation expr. Bootstrapped/regtested on x86_64-pc-linux-gnu{-m32,} and sde{-m32,} OK for master? gcc/ChangeLog: * config/i386/i386-expand.c (ix86_expand_vec_perm): Convert HFmode input ope