[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