[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 21 11:50:39 PDT 2022


craig.topper added inline comments.
Herald added a subscriber: StephenFan.


================
Comment at: llvm/lib/Target/RISCV/RISCVISelLowering.cpp:478
+
+      setOperationAction(ISD::EXPERIMENTAL_VP_STRIDED_LOAD, MVT::i32, Custom);
+      setOperationAction(ISD::EXPERIMENTAL_VP_STRIDED_STORE, MVT::i32, Custom);
----------------
Why do we need custom legalization for i32 on rv64? Shouldn't we have a generic PromoteIntOp for this?


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

https://reviews.llvm.org/D121113



More information about the llvm-commits mailing list