[PATCH] D53811: [MSan] another take at instrumenting inline assembly - now with calls
Alexander Potapenko via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 30 02:45:23 PDT 2018
glider added inline comments.
================
Comment at: lib/Transforms/Instrumentation/MemorySanitizer.cpp:3121
if (Call->isInlineAsm()) {
- if (ClHandleAsmConservative)
+ if (ClHandleAsmConservative && InsertChecks)
visitAsmInstruction(I);
----------------
eugenis wrote:
> Why test for InsertCheck here?
> It is done in insertShadowCheck already, and unpoisoning of stores should be done even in blacklisted functions.
Fair enough, will remove.
https://reviews.llvm.org/D53811
More information about the llvm-commits
mailing list