[PATCH] D137530: [RISCV] Splat scalar to be of length VL instead of 1 for reductions

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Nov 10 22:42:33 PST 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:5724
   SDValue IdentitySplat =
       lowerScalarSplat(SDValue(), NeutralElem, DAG.getConstant(1, DL, XLenVT),
                        M1VT, DL, DAG, Subtarget);
----------------
Move this into an `else`. We shouldn't create nodes if they are going to end up dead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D137530



More information about the llvm-commits mailing list