[PATCH] D113489: [AArch64][SVE] Instcombine SVE LD1/ST1 to stock LLVM IR

[Peter Waller via Phabricator via llvm-commits]

peterwaller-arm added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:731
+  Value *PtrOp = II.getOperand(1);
+  Type *VecTyPtr = II.getType()->getPointerTo();
+  IRBuilder<> Builder(II.getContext());
----------------
VecPtrTy -- the Ty conventionally goes at the end.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp:783
+  Value *PtrOp = II.getOperand(2);
+  Type *VecTyPtr = VecOp->getType()->getPointerTo();
+  IRBuilder<> Builder(II.getContext());
----------------
`s/VecTyPtr/VecPtrTy/`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D113489