[PATCH] D68400: [NFC][TTI] Add Alignment for isLegalMasked[Load/Store]

Dave Green via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 7 14:57:24 PDT 2019


dmgreen added inline comments.


================
Comment at: lib/Target/X86/X86TargetTransformInfo.h:189
+  bool isLegalMaskedLoad(Type *DataType, unsigned Alignment);
   bool isLegalMaskedStore(Type *DataType);
   bool isLegalNTLoad(Type *DataType, Align Alignment);
----------------
", unsigned Alignment" :)

Also the isLegalNTLoad below use the Align. I think they should be fairly simple to use (but haven't used them myself yet, just fixed merge conflicts).


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

https://reviews.llvm.org/D68400





More information about the llvm-commits mailing list