[PATCH] D61764: [LV] Suppress vectorization in some nontemporal cases

[Simon Pilgrim via Phabricator via llvm-commits]

RKSimon accepted this revision.
RKSimon added a comment.

Thanks @wristow LGTM with one minor



================
Comment at: include/llvm/Analysis/TargetTransformInfoImpl.h:240
+  bool isLegalNTLoad(Type *DataType, unsigned Alignment) {
+    return isLegalNTStore(DataType, Alignment);
+  }
----------------
Might be safer to duplicate the isLegalNTStore code depending on how well targets override both/either of the calls?


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

https://reviews.llvm.org/D61764