On Mon, Jan 4, 2021 at 4:49 PM Jakub Jelinek <ja...@redhat.com> wrote: > > On Mon, Jan 04, 2021 at 01:56:44PM +0800, Hongtao Liu via Gcc-patches wrote: > > +(define_insn_and_split "*sse2_pmovskb_zexthisi" > > + [(set (match_operand:SI 0 "register_operand") > > + (zero_extend:SI (subreg:HI (unspec:SI > > + [(match_operand:V16QI 1 "register_operand")] > > + UNSPEC_MOVMSK) 0)))] > > Also, please fix up formatting. Should be: > (zero_extend:SI > (subreg:HI > (unspec:SI > [(match_operand:V16QI 1 "register_operand")] > UNSPEC_MOVMSK) 0)))] > I think. > > Jakub >
Yes, thanks for the review both, and happy new year! -- BR, Hongtao