[RISCV] Use MutableArrayRef instead of SmallVectorImpl&. NFC

[topperc]

We're only going to modify existing items, not add or remove any elements to the vector.

[llvmbot]

@llvm/pr-subscribers-backend-risc-v

Author: Craig Topper (topperc)

Changes

We're only going to modify exiting items, not add or remove any elements to the vector.

---

Full diff: https://github.com/llvm/llvm-project/pull/159651.diff

1 Files Affected:

• (modified) llvm/lib/Target/RISCV/RISCVISelLowering.cpp (+2-2)

diff --git a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
index d5240f971f815..71e9bf2a7882f 100644
--- a/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
+++ b/llvm/lib/Target/RISCV/RISCVISelLowering.cpp
@@ -10719,7 +10719,7 @@ static SDValue lowerCttzElts(SDNode *N, SelectionDAG &DAG,
 }
 
 static inline void promoteVCIXScalar(const SDValue &Op,
-                                     SmallVectorImpl<SDValue> &Operands,
+                                     MutableArrayRef<SDValue> Operands,
                                      SelectionDAG &DAG) {
   const RISCVSubtarget &Subtarget =
       DAG.getMachineFunction().getSubtarget<RISCVSubtarget>();
@@ -10755,7 +10755,7 @@ static inline void promoteVCIXScalar(const SDValue &Op,
 }
 
 static void processVCIXOperands(SDValue &OrigOp,
-                                SmallVectorImpl<SDValue> &Operands,
+                                MutableArrayRef<SDValue> Operands,
                                 SelectionDAG &DAG) {
   promoteVCIXScalar(OrigOp, Operands, DAG);
   const RISCVSubtarget &Subtarget =

[mshockwave]

LGTM
There is a typo in your PR description though: "exiting" -> "existing"

[topperc]

LGTM There is a typo in your PR description though: "exiting" -> "existing"

Thanks. Fixed now.

[4vtomat]

LGTM~