[llvm] d6b224e - [RISCV] Rename isAllUsesReadW to hasAllWUsers in RISCVSExtWRemoval. NFC

[Craig Topper via llvm-commits]

Author: Craig Topper
Date: 2022-11-04T17:04:44-07:00
New Revision: d6b224edd3f0c8f925d2435b487e5ea4a1ccc29d

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

LOG: [RISCV] Rename isAllUsesReadW to hasAllWUsers in RISCVSExtWRemoval. NFC

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVSExtWRemoval.cpp b/llvm/lib/Target/RISCV/RISCVSExtWRemoval.cpp
index 14115982313a3..14d7afb512e1a 100644
--- a/llvm/lib/Target/RISCV/RISCVSExtWRemoval.cpp
+++ b/llvm/lib/Target/RISCV/RISCVSExtWRemoval.cpp
@@ -73,8 +73,7 @@ static void addUses(const MachineInstr &MI,
 // returns true if all uses of OrigMI only depend on the lower word of its
 // output, so we can transform OrigMI to the corresponding W-version.
 // TODO: handle multiple interdependent transformations
-static bool isAllUsesReadW(const MachineInstr &OrigMI,
-                           MachineRegisterInfo &MRI) {
+static bool hasAllWUsers(const MachineInstr &OrigMI, MachineRegisterInfo &MRI) {
 
   SmallPtrSet<const MachineInstr *, 4> Visited;
   SmallVector<const MachineInstr *, 4> Worklist;
@@ -258,7 +257,7 @@ static bool isSignExtendingOpW(MachineInstr &MI, MachineRegisterInfo &MRI,
   case RISCV::ADDI:
     if (MI.getOperand(1).isReg() && MI.getOperand(1).getReg() == RISCV::X0)
       return true;
-    if (isAllUsesReadW(MI, MRI)) {
+    if (hasAllWUsers(MI, MRI)) {
       // transform to ADDIW
       FixableDef.insert(&MI);
       return true;
@@ -286,7 +285,7 @@ static bool isSignExtendingOpW(MachineInstr &MI, MachineRegisterInfo &MRI,
   case RISCV::LWU:
   case RISCV::MUL:
   case RISCV::SUB:
-    if (isAllUsesReadW(MI, MRI)) {
+    if (hasAllWUsers(MI, MRI)) {
       FixableDef.insert(&MI);
       return true;
     }