[PATCH] D74581: [llvm][CodeGen][aarch64] Add contiguous prefetch intrinsics for SVE.

Francesco Petrogalli via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 21 12:31:28 PST 2020


fpetrogalli marked 4 inline comments as done.
fpetrogalli added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64ISelDAGToDAG.cpp:4414
 
+namespace {
+/// When \p PredVT is a scalable vector predicate in the form
----------------
sdesmalen wrote:
> nit: Any reason you're using an anonymous namespace over `static` for these functions?
I should have really used `static` to adhere to the style of the rest of the file... Done.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74581





More information about the llvm-commits mailing list