Re: [x86 SSE PATCH] Some AVX512 ternlog expansion refinements (take #2)

2024-07-11 Thread Hongtao Liu
emit_move_insn for ~a, ~b and ~c cases. > > > Thanks again, > Roger > > > -Original Message----- > > From: Hongtao Liu > > Sent: 08 July 2024 02:55 > > To: Roger Sayle > > Cc: gcc-patches@gcc.gnu.org; Uros Bizjak > > Subject: Re: [x86 SSE PA

[x86 SSE PATCH] Some AVX512 ternlog expansion refinements (take #2)

2024-07-11 Thread Roger Sayle
_VECTOR_P instead of equality against GET_CODE. Use gen_reg_rtx and emit_move_insn for ~a, ~b and ~c cases. Thanks again, Roger > -Original Message- > From: Hongtao Liu > Sent: 08 July 2024 02:55 > To: Roger Sayle > Cc: gcc-patches@gcc.gnu.org; Uros Bizjak > Su

Re: [x86 SSE PATCH] Some AVX512 ternlog expansion refinements.

2024-07-07 Thread Hongtao Liu
On Sun, Jul 7, 2024 at 5:00 PM Roger Sayle wrote: > > > Hi Hongtao, > This should address concerns about the remaining use of force_reg. > 51@@ -25793,15 +25792,20 @@ ix86_expand_ternlog_binop (enum rtx_code code, machine_mode mode, 52 if (GET_MODE (op1) != mode) 53 op1 = gen_lowpart (mod

[x86 SSE PATCH] Some AVX512 ternlog expansion refinements.

2024-07-07 Thread Roger Sayle
Hi Hongtao, This should address concerns about the remaining use of force_reg. This patch replaces the call to force_reg in ix86_expand_ternlog_binop with gen_reg_rtx and emit_move_insn, the last place where force_reg may be called (indirectly) from ix86_expand_ternlog. This patch also cleans up