[PATCH] D131687: [Clang]Replace aarch64_sve_ldN intrinsic by aarch64_sve_ldN.sret

[Sander de Smalen via Phabricator via llvm-commits]

sdesmalen accepted this revision.
sdesmalen added a comment.
This revision is now accepted and ready to land.

LGTM with nit addressed, thanks @CarolineConcatto!



================
Comment at: clang/lib/CodeGen/CGBuiltin.cpp:8866
+  Value *Call = Builder.CreateCall(F, {Predicate, BasePtr});
+  unsigned MinElts = VTy->getElementCount().getKnownMinValue();
+  Value *Ret = llvm::PoisonValue::get(RetTy);
----------------
nit: `s/VTy->getElementCount().getKnownMinValue()/VTy->getMinNumElements()/`


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131687