[llvm] 8e02428 - [RISCV] Minor style cleanups in post ISEL combines

[Philip Reames via llvm-commits]

Author: Philip Reames
Date: 2023-07-18T12:26:36-07:00
New Revision: 8e024283bd3873891e4b59ee3f3a11589805df97

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

LOG: [RISCV] Minor style cleanups in post ISEL combines

Added: 
    

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

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp b/llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp
index 4ee2aec191734f..f7b37eda8bfaac 100644
--- a/llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp
+++ b/llvm/lib/Target/RISCV/RISCVISelDAGToDAG.cpp
@@ -301,15 +301,11 @@ void RISCVDAGToDAGISel::addVectorLoadStoreOperands(
   // none of the others do.  All have passthru operands.  For our pseudos,
   // all loads have policy operands.
   if (IsLoad) {
-    if (IsMasked) {
-      uint64_t Policy = Node->getConstantOperandVal(CurOp++);
-      SDValue PolicyOp = CurDAG->getTargetConstant(Policy, DL, XLenVT);
-      Operands.push_back(PolicyOp);
-    } else {
-      uint64_t Policy = RISCVII::MASK_AGNOSTIC;
-      SDValue PolicyOp = CurDAG->getTargetConstant(Policy, DL, XLenVT);
-      Operands.push_back(PolicyOp);
-    }
+    uint64_t Policy = RISCVII::MASK_AGNOSTIC;
+    if (IsMasked)
+      Policy = Node->getConstantOperandVal(CurOp++);
+    SDValue PolicyOp = CurDAG->getTargetConstant(Policy, DL, XLenVT);
+    Operands.push_back(PolicyOp);
   }
 
   Operands.push_back(Chain); // Chain.
@@ -3178,11 +3174,9 @@ bool RISCVDAGToDAGISel::doPeepholeMaskedRVV(SDNode *N) {
     return false;
 
   unsigned MaskOpIdx = I->MaskOpIdx;
-
   if (!usesAllOnesMask(N, MaskOpIdx))
     return false;
 
-
   // There are two classes of pseudos in the table - compares and
   // everything else.  See the comment on RISCVMaskedPseudo for details.
   const unsigned Opc = I->UnmaskedPseudo;