[PATCH] D53811: [MSan] another take at instrumenting inline assembly - now with calls

[Evgenii Stepanov via Phabricator via llvm-commits]

eugenis added inline comments.


================
Comment at: lib/Transforms/Instrumentation/MemorySanitizer.cpp:3121
       if (Call->isInlineAsm()) {
-        if (ClHandleAsmConservative)
+        if (ClHandleAsmConservative && InsertChecks)
           visitAsmInstruction(I);
----------------
Why test for InsertCheck here?
It is done in insertShadowCheck already, and unpoisoning of stores should be done even in blacklisted functions.


https://reviews.llvm.org/D53811