Re: [PATCH] RISC-V: Fix RISCV_FUSE_ZEXTWS fusion condition

[Jeff Law]

On 12/20/23 01:39, Wang Pengcheng wrote:
The condition is RISCV_FUSE_ZEXTH, which is a mistake.

gcc/ChangeLog:

* config/riscv/riscv.cc (riscv_macro_fusion_pair_p): Fix condition.
I've pushed this to the trunk.  Attached is the actual patch committed 
which also fixes formatting of that code.

Thanks again!
jeffcommit eef65d60a8bb2e9328fd9d2b6cd869618be4f08e
Author: Wang Pengcheng <wangpengcheng...@bytedance.com>
Date:   Wed Dec 20 10:32:59 2023 -0700

    [PATCH] RISC-V: Fix RISCV_FUSE_ZEXTWS fusion condition
    
    The condition is RISCV_FUSE_ZEXTH, which is a mistake.
    
    gcc/ChangeLog:
    
            * config/riscv/riscv.cc (riscv_macro_fusion_pair_p): Fix condition.

diff --git a/gcc/config/riscv/riscv.cc b/gcc/config/riscv/riscv.cc
index d9b45f17a1b..c6784a22127 100644
--- a/gcc/config/riscv/riscv.cc
+++ b/gcc/config/riscv/riscv.cc
@@ -8096,8 +8096,9 @@ riscv_macro_fusion_pair_p (rtx_insn *prev, rtx_insn *curr)
   if (!riscv_macro_fusion_p ())
     return false;
 
-  if (simple_sets_p && (riscv_fusion_enabled_p (RISCV_FUSE_ZEXTW) ||
-                       riscv_fusion_enabled_p (RISCV_FUSE_ZEXTH)))
+  if (simple_sets_p
+      && (riscv_fusion_enabled_p (RISCV_FUSE_ZEXTW)
+         || riscv_fusion_enabled_p (RISCV_FUSE_ZEXTWS)))
     {
       /* We are trying to match the following:
           prev (slli) == (set (reg:DI rD)