[PATCH] D13264: Disabling speculative loads under asan.

Kostya Serebryany via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 12 15:16:42 PDT 2015


kcc added a comment.

This will need an llvm-ir .lit test (in addition to http://reviews.llvm.org/D13265)


================
Comment at: lib/Analysis/ValueTracking.cpp:3191
@@ +3190,3 @@
+        LI->getParent()->getParent()->hasFnAttribute(Attribute::SanitizeThread)
+        ||
+        // Speculative load may load data from dirty regions.
----------------
clang-format, please


http://reviews.llvm.org/D13264





More information about the llvm-commits mailing list