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

Warren Ristow via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 13 19:37:25 PDT 2019


wristow marked an inline comment as done.
wristow added inline comments.


================
Comment at: include/llvm/Analysis/TargetTransformInfoImpl.h:240
+  bool isLegalNTLoad(Type *DataType, unsigned Alignment) {
+    return isLegalNTStore(DataType, Alignment);
+  }
----------------
RKSimon wrote:
> Might be safer to duplicate the isLegalNTStore code depending on how well targets override both/either of the calls?
Will do.  (I'm on vacation right now, but will wrap this up first hing next week.)


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

https://reviews.llvm.org/D61764





More information about the llvm-commits mailing list