commit d93f91fc467beb3da99a43fc1874f1dbcaf250c4
author Philip Reames <preames@rivosinc.com> Mon Jul 21 12:14:26 2025 -0700
committer Philip Reames <listmail@philipreames.com> Mon Jul 21 12:24:51 2025 -0700
tree 5bd01f507d535eb392239652c6e0a0f651125698
parent fe267860c1206b0622f5aee8fe8a04040fcecbf7

[RISCV][IA] Prefer switch over intrinsic ID instead of if-chain [nfc]

llvm/lib/Target/RISCV/RISCVInterleavedAccess.cpp

diff --git a/llvm/lib/Target/RISCV/RISCVInterleavedAccess.cpp b/llvm/lib/Target/RISCV/RISCVInterleavedAccess.cpp
index 6de870c..7d1f789 100644
--- a/llvm/lib/Target/RISCV/RISCVInterleavedAccess.cpp
+++ b/llvm/lib/Target/RISCV/RISCVInterleavedAccess.cpp
@@ -131,10 +131,14 @@
                                    : Constant::getAllOnesValue(XLenTy);
     return true;
   }
-  if (auto *VPLdSt = dyn_cast<VPIntrinsic>(I)) {
-    assert((VPLdSt->getIntrinsicID() == Intrinsic::vp_load ||
-            VPLdSt->getIntrinsicID() == Intrinsic::vp_store) &&
-           "Unexpected intrinsic");
+
+  auto *II = cast<IntrinsicInst>(I);
+  switch (II->getIntrinsicID()) {
+  default:
+    llvm_unreachable("Unsupported intrinsic type");
+  case Intrinsic::vp_load:
+  case Intrinsic::vp_store: {
+    auto *VPLdSt = cast<VPIntrinsic>(I);
     Ptr = VPLdSt->getMemoryPointerParam();
     Alignment = VPLdSt->getPointerAlignment().value_or(
         DL.getABITypeAlign(VTy->getElementType()));
@@ -151,21 +155,21 @@
     VL = Builder.CreateZExt(Builder.CreateExactUDiv(WideEVL, FactorC), XLenTy);
     return true;
   }
-  auto *II = cast<IntrinsicInst>(I);
-  assert(II->getIntrinsicID() == Intrinsic::masked_load &&
-         "Unexpected intrinsic");
-  Ptr = II->getOperand(0);
-  Alignment = cast<ConstantInt>(II->getArgOperand(1))->getAlignValue();
+  case Intrinsic::masked_load: {
+    Ptr = II->getOperand(0);
+    Alignment = cast<ConstantInt>(II->getArgOperand(1))->getAlignValue();
 
-  if (!isa<UndefValue>(II->getOperand(3)))
-    return false;
+    if (!isa<UndefValue>(II->getOperand(3)))
+      return false;
 
-  assert(Mask && "masked.load needs a mask!");
+    assert(Mask && "masked.load needs a mask!");
 
-  VL = isa<FixedVectorType>(VTy)
-           ? Builder.CreateElementCount(XLenTy, VTy->getElementCount())
-           : Constant::getAllOnesValue(XLenTy);
-  return true;
+    VL = isa<FixedVectorType>(VTy)
+             ? Builder.CreateElementCount(XLenTy, VTy->getElementCount())
+             : Constant::getAllOnesValue(XLenTy);
+    return true;
+  }
+  }
 }
 
 /// Lower an interleaved load into a vlsegN intrinsic.