[llvm] [RISCV] Prefer vrgatherei16 for shuffles (PR #66291)

[Craig Topper via llvm-commits]

================
@@ -4651,6 +4651,15 @@ static SDValue lowerVECTOR_SHUFFLE(SDValue Op, SelectionDAG &DAG,
     IndexVT = IndexVT.changeVectorElementType(MVT::i16);
   }
 
+  // If the mask allows, we can do all the index computation in 16 bits.  This
+  // requires less work and less register pressure at high LMUL, and creates
+  // smaller constants which may be cheaper to materialize.
+  if (IndexVT.getScalarType().bitsGT(MVT::i16) && isUInt<16>(NumElts * 2) &&
----------------
topperc wrote:

Why `NumElts * 2`? Shouldn't it be `NumElts - 1`?

https://github.com/llvm/llvm-project/pull/66291