[PATCH] D121113: [RISCV][VP] Custom lower VP_STRIDED_LOAD and VP_STRIDED_STORE

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 7 10:27:18 PST 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:476
+
+    setOperationAction(ISD::EXPERIMENTAL_VP_STRIDED_LOAD, MVT::i8, Custom);
+    setOperationAction(ISD::EXPERIMENTAL_VP_STRIDED_LOAD, MVT::i16, Custom);
----------------
Why do we need to Custom legalize the narrow scalar types? Shouldn't they be promoted by generic type legalization PromoteIntOp?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121113



More information about the llvm-commits mailing list