[PATCH] D131964: [AArch64] Add support to loop vectorization for non temporal loads

Zain Jaffal via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 3 02:05:09 PDT 2022


zjaffal added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h:322
   bool isLegalNTStore(Type *DataType, Align Alignment) {
     // NOTE: The logic below is mostly geared towards LV, which calls it with
     //       vectors with 2 elements. We might want to improve that, if other
----------------
fhahn wrote:
> This should be also moved to `isLegalNTStoreLoad`
This line only or the whole comment block ?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D131964



More information about the llvm-commits mailing list