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

Filipe Cabecinhas via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Dec 14 10:51:46 PST 2016

filcab planned changes to this revision.
filcab 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.
kcc wrote:
> I'd expect this to be inside if (ClOpt && ClOptSameTemp)
> No? 
Ah! Right. Will submit new version.


More information about the llvm-commits mailing list