Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-06-01 Thread Segher Boessenkool
> PR target/100711 > * simplify-rtx.c (simplify_unary_operation_1): > Simplify (vec_duplicate (not op)) to (not (vec_duplicate op)). This is not a simplification. If we want to do this we need to document this canonicalisation (in md.texi, "Insn Canonicalizations"). > +/* P

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-06-01 Thread Segher Boessenkool
On Tue, Jun 01, 2021 at 04:32:42PM +0800, Hongtao Liu wrote: [ no attachment to reply to ] Please send this with either the patch actually inline, or as attachment with content-disposition inline, no encoding, and a valid text mimetype. So that people can see it, also on the archives, and actual

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-06-01 Thread Hongtao Liu via Gcc-patches
On Wed, May 26, 2021 at 1:17 PM Hongtao Liu wrote: > > On Wed, May 26, 2021 at 12:12 PM Andrew Pinski wrote: > > > > On Tue, May 25, 2021 at 6:17 PM Hongtao Liu wrote: > > > > > > Update patch: > > > The new patch simplify (vec_duplicate (not (nonimmedaite_operand))) > > > to (not (vec_duplica

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-25 Thread Hongtao Liu via Gcc-patches
On Wed, May 26, 2021 at 12:12 PM Andrew Pinski wrote: > > On Tue, May 25, 2021 at 6:17 PM Hongtao Liu wrote: > > > > Update patch: > > The new patch simplify (vec_duplicate (not (nonimmedaite_operand))) > > to (not (vec_duplicate (nonimmedaite_operand))). This is not a > > straightforward simpl

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-25 Thread Andrew Pinski via Gcc-patches
On Tue, May 25, 2021 at 6:17 PM Hongtao Liu wrote: > > Update patch: > The new patch simplify (vec_duplicate (not (nonimmedaite_operand))) > to (not (vec_duplicate (nonimmedaite_operand))). This is not a > straightforward simplification, just adding some tendency to pull not > out of vec_duplica

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-25 Thread Hongtao Liu via Gcc-patches
Update patch: The new patch simplify (vec_duplicate (not (nonimmedaite_operand))) to (not (vec_duplicate (nonimmedaite_operand))). This is not a straightforward simplification, just adding some tendency to pull not out of vec_duplicate. For i386, it will enable below opt from notl

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-24 Thread Hongtao Liu via Gcc-patches
On Tue, May 25, 2021 at 2:29 PM Andrew Pinski wrote: > > On Mon, May 24, 2021 at 11:23 PM Hongtao Liu wrote: > > > > On Tue, May 25, 2021 at 2:11 PM Andrew Pinski wrote: > > > > > > On Mon, May 24, 2021 at 11:03 PM Hongtao Liu via Gcc-patches > > > wrote: > > > > > > > > Hi: > > > > This patc

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-24 Thread Andrew Pinski via Gcc-patches
On Mon, May 24, 2021 at 11:23 PM Hongtao Liu wrote: > > On Tue, May 25, 2021 at 2:11 PM Andrew Pinski wrote: > > > > On Mon, May 24, 2021 at 11:03 PM Hongtao Liu via Gcc-patches > > wrote: > > > > > > Hi: > > > This patch is about to do transformation like below. > > > Bootstrapped and regte

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-24 Thread Hongtao Liu via Gcc-patches
On Tue, May 25, 2021 at 2:11 PM Andrew Pinski wrote: > > On Mon, May 24, 2021 at 11:03 PM Hongtao Liu via Gcc-patches > wrote: > > > > Hi: > > This patch is about to do transformation like below. > > Bootstrapped and regtested on x86_64-linux-gnu{-m32,}. > > Ok for trunk? > > > > from > >

Re: [PATCH][i386] Split not+broadcast+pand to broadcast+pandn.

2021-05-24 Thread Andrew Pinski via Gcc-patches
On Mon, May 24, 2021 at 11:03 PM Hongtao Liu via Gcc-patches wrote: > > Hi: > This patch is about to do transformation like below. > Bootstrapped and regtested on x86_64-linux-gnu{-m32,}. > Ok for trunk? > > from > notl%edi > vpbroadcastd%edi, %xmm0 > vpand