[llvm] 44feae8 - [RISCV][VLOPT] Mark some methods + arguments as const. NFC

[Luke Lau via llvm-commits]

Author: Luke Lau
Date: 2025-02-20T17:37:27+08:00
New Revision: 44feae869570201e9920c26b151ce7ce24f0418d

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

LOG: [RISCV][VLOPT] Mark some methods + arguments as const. NFC

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVVLOptimizer.cpp b/llvm/lib/Target/RISCV/RISCVVLOptimizer.cpp
index 8e952f5184098..ba2fcdc6d0670 100644
--- a/llvm/lib/Target/RISCV/RISCVVLOptimizer.cpp
+++ b/llvm/lib/Target/RISCV/RISCVVLOptimizer.cpp
@@ -50,11 +50,12 @@ class RISCVVLOptimizer : public MachineFunctionPass {
   StringRef getPassName() const override { return PASS_NAME; }
 
 private:
-  std::optional<MachineOperand> getMinimumVLForUser(MachineOperand &UserOp);
+  std::optional<MachineOperand>
+  getMinimumVLForUser(const MachineOperand &UserOp) const;
   /// Returns the largest common VL MachineOperand that may be used to optimize
   /// MI. Returns std::nullopt if it failed to find a suitable VL.
-  std::optional<MachineOperand> checkUsers(MachineInstr &MI);
-  bool tryReduceVL(MachineInstr &MI);
+  std::optional<MachineOperand> checkUsers(const MachineInstr &MI) const;
+  bool tryReduceVL(MachineInstr &MI) const;
   bool isCandidate(const MachineInstr &MI) const;
 
   /// For a given instruction, records what elements of it are demanded by
@@ -1152,8 +1153,8 @@ static bool isSupportedInstr(const MachineInstr &MI) {
 }
 
 /// Return true if MO is a vector operand but is used as a scalar operand.
-static bool isVectorOpUsedAsScalarOp(MachineOperand &MO) {
-  MachineInstr *MI = MO.getParent();
+static bool isVectorOpUsedAsScalarOp(const MachineOperand &MO) {
+  const MachineInstr *MI = MO.getParent();
   const RISCVVPseudosTable::PseudoInfo *RVV =
       RISCVVPseudosTable::getPseudoInfo(MI->getOpcode());
 
@@ -1261,7 +1262,7 @@ bool RISCVVLOptimizer::isCandidate(const MachineInstr &MI) const {
 }
 
 std::optional<MachineOperand>
-RISCVVLOptimizer::getMinimumVLForUser(MachineOperand &UserOp) {
+RISCVVLOptimizer::getMinimumVLForUser(const MachineOperand &UserOp) const {
   const MachineInstr &UserMI = *UserOp.getParent();
   const MCInstrDesc &Desc = UserMI.getDesc();
 
@@ -1282,7 +1283,7 @@ RISCVVLOptimizer::getMinimumVLForUser(MachineOperand &UserOp) {
   if (UserOp.isTied()) {
     assert(UserOp.getOperandNo() == UserMI.getNumExplicitDefs() &&
            RISCVII::isFirstDefTiedToFirstUse(UserMI.getDesc()));
-    auto DemandedVL = DemandedVLs[&UserMI];
+    auto DemandedVL = DemandedVLs.lookup(&UserMI);
     if (!DemandedVL || !RISCV::isVLKnownLE(*DemandedVL, VLOp)) {
       LLVM_DEBUG(dbgs() << "    Abort because user is passthru in "
                            "instruction with demanded tail\n");
@@ -1304,7 +1305,7 @@ RISCVVLOptimizer::getMinimumVLForUser(MachineOperand &UserOp) {
 
   // If we know the demanded VL of UserMI, then we can reduce the VL it
   // requires.
-  if (auto DemandedVL = DemandedVLs[&UserMI]) {
+  if (auto DemandedVL = DemandedVLs.lookup(&UserMI)) {
     assert(isCandidate(UserMI));
     if (RISCV::isVLKnownLE(*DemandedVL, VLOp))
       return DemandedVL;
@@ -1313,7 +1314,8 @@ RISCVVLOptimizer::getMinimumVLForUser(MachineOperand &UserOp) {
   return VLOp;
 }
 
-std::optional<MachineOperand> RISCVVLOptimizer::checkUsers(MachineInstr &MI) {
+std::optional<MachineOperand>
+RISCVVLOptimizer::checkUsers(const MachineInstr &MI) const {
   std::optional<MachineOperand> CommonVL;
   SmallSetVector<MachineOperand *, 8> Worklist;
   for (auto &UserOp : MRI->use_operands(MI.getOperand(0).getReg()))
@@ -1386,7 +1388,7 @@ std::optional<MachineOperand> RISCVVLOptimizer::checkUsers(MachineInstr &MI) {
   return CommonVL;
 }
 
-bool RISCVVLOptimizer::tryReduceVL(MachineInstr &MI) {
+bool RISCVVLOptimizer::tryReduceVL(MachineInstr &MI) const {
   LLVM_DEBUG(dbgs() << "Trying to reduce VL for " << MI << "\n");
 
   unsigned VLOpNum = RISCVII::getVLOpNum(MI.getDesc());
@@ -1399,7 +1401,7 @@ bool RISCVVLOptimizer::tryReduceVL(MachineInstr &MI) {
     return false;
   }
 
-  auto CommonVL = DemandedVLs[&MI];
+  auto CommonVL = DemandedVLs.lookup(&MI);
   if (!CommonVL)
     return false;