[llvm] 971bfbe - [RISCV][NFC] Add NumTransformedToNonWInstrs statistic to RISCVOptWInstrs extend debug printing

Alex Bradbury via llvm-commits llvm-commits at lists.llvm.org
Sun Jul 20 01:54:59 PDT 2025


Author: Alex Bradbury
Date: 2025-07-20T09:54:10+01:00
New Revision: 971bfbead2a1ed1c8b44f5b7a3003d91f46cff60

URL: https://github.com/llvm/llvm-project/commit/971bfbead2a1ed1c8b44f5b7a3003d91f46cff60
DIFF: https://github.com/llvm/llvm-project/commit/971bfbead2a1ed1c8b44f5b7a3003d91f46cff60.diff

LOG: [RISCV][NFC] Add NumTransformedToNonWInstrs statistic to RISCVOptWInstrs extend debug printing

RISCVOptWInstrs has a NumTransformedToWInstrs statistic, but didn't have
one for the W=>Non-W transform done by stripWSuffixes. It also didn't do
debug printing of the transformation. This patch addresses both issues.

Reviewed as part of <https://github.com/llvm/llvm-project/pull/149071>,
but landing separately.

Added: 
    

Modified: 
    llvm/lib/Target/RISCV/RISCVOptWInstrs.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVOptWInstrs.cpp b/llvm/lib/Target/RISCV/RISCVOptWInstrs.cpp
index 28d64031f8bcb..1964ef43beda7 100644
--- a/llvm/lib/Target/RISCV/RISCVOptWInstrs.cpp
+++ b/llvm/lib/Target/RISCV/RISCVOptWInstrs.cpp
@@ -48,6 +48,8 @@ using namespace llvm;
 STATISTIC(NumRemovedSExtW, "Number of removed sign-extensions");
 STATISTIC(NumTransformedToWInstrs,
           "Number of instructions transformed to W-ops");
+STATISTIC(NumTransformedToNonWInstrs,
+          "Number of instructions transformed to non-W-ops");
 
 static cl::opt<bool> DisableSExtWRemoval("riscv-disable-sextw-removal",
                                          cl::desc("Disable removal of sext.w"),
@@ -739,7 +741,10 @@ bool RISCVOptWInstrs::stripWSuffixes(MachineFunction &MF,
       }
 
       if (hasAllWUsers(MI, ST, MRI)) {
+        LLVM_DEBUG(dbgs() << "Replacing " << MI);
         MI.setDesc(TII.get(Opc));
+        LLVM_DEBUG(dbgs() << "     with " << MI);
+        ++NumTransformedToNonWInstrs;
         MadeChange = true;
       }
     }


        


More information about the llvm-commits mailing list