[PATCH] D119110: [RISCV] support vwmulsu_vx when one input is a scalar splat

[Fraser Cormack via Phabricator via llvm-commits]

frasercrmck added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:7625
 
-    if (IsSignExt) {
+    if (IsSignExt && ISD::isZEXTLoad(Op1.getNode())) {
+      APInt Mask = APInt::getBitsSetFrom(ScalarBits, NarrowSize);
----------------
The description says that we're now supporting scalar splats but AFAICT this will only work for zero-extending loads? Feels like maybe the testing you're adding is too narrowly-focused and dependent on the `load`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D119110