[PATCH] D66688: [LoopVectorize] Leverage speculation safety to avoid masked.loads

Philip Reames via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 6 15:51:52 PDT 2019


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


================
Comment at: lib/Transforms/Vectorize/LoopVectorizationLegality.cpp:976
+    // Speculative load may load data from dirty regions.
+    F.hasFnAttribute(Attribute::SanitizeAddress) ||
+    F.hasFnAttribute(Attribute::SanitizeHWAddress);
----------------
xbolva00 wrote:
> Also SanitizeMemory?
If so, as a separate change.  This exactly matches the list in ValueTracking.  


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

https://reviews.llvm.org/D66688





More information about the llvm-commits mailing list