[PATCH] D121784: [VP] Add splitting for VP_STRIDED_STORE and VP_STRIDED_LOAD

Craig Topper via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 12:20:16 PDT 2022


craig.topper added inline comments.


================
Comment at: llvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp:1936
+        MachinePointerInfo(SLD->getPointerInfo().getAddrSpace()), HiMemVT,
+        SLD->getOriginalAlign(), MachineMemOperand::MOLoad, SLD->getAAInfo(),
+        SLD->getRanges(), SLD->isExpandingLoad());
----------------
This will create a MachineMemOperand with size based on HiMemVT, but I don't think that's correct. We need UnknownSize.


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

https://reviews.llvm.org/D121784



More information about the llvm-commits mailing list