[PATCH] D52497: [MSan] Add meaningful debug locations to [K]MSan hooks poisoning local variables

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 25 09:06:31 PDT 2018


aprantl requested changes to this revision.
aprantl added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/Transforms/Instrumentation/MemorySanitizer.cpp:3282
+        // Check for DW_OP_deref to make sure this intrinsic is referencing
+        // a variable, not a function argument.
+        if ((isa<DbgValueInst>(DVI) &&
----------------
See my reply to llvm-dev; this is not the right condition to check for.


Repository:
  rL LLVM

https://reviews.llvm.org/D52497





More information about the llvm-commits mailing list