[PATCH] D27625: [asan] Don't skip instrumentation of masked load/store unless we've seen a full load/store on that pointer.

Kostya Serebryany via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 10:50:49 PST 2016


kcc added inline comments.


================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:2072
+        if (MaybeMask) {
+          if (TempsToInstrument.count(Addr))
+            continue;  // We've seen this (whole) temp in the current BB.
----------------
I'd expect this to be inside if (ClOpt && ClOptSameTemp)
No? 


https://reviews.llvm.org/D27625





More information about the llvm-commits mailing list