[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]

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.


https://reviews.llvm.org/D27625