[PATCH] D157117: [RISCV] Use vmerge for INSERT_VECTOR_ELT w/small constant indices

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 9 03:44:33 PDT 2023


luke added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:6939
+    if (isa<ConstantSDNode>(Idx) && cast<ConstantSDNode>(Idx)->getZExtValue() <= 4 &&
+        VecVT.isFixedLengthVector()) {
+      SmallVector<SDValue> Ops;
----------------
Could we still use vmv.v.i on scalable vectors in theory? Since Idx isn't scaled by vscale for insert_vector_elt.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D157117/new/

https://reviews.llvm.org/D157117



More information about the llvm-commits mailing list