[PATCH] D41222: Handle previously ASAN-instrumented IR gracefully when ASAN re-invoked
Vitaly Buka via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Thu Dec 14 10:53:18 PST 2017
vitalybuka added inline comments.
================
Comment at: lib/Transforms/Instrumentation/AddressSanitizer.cpp:2422
+ // Skip modules which already have ASan instrumentation.
+ if (isAlreadyAsanInstrumented(*F.getParent()))
+ return false;
----------------
Simple if statements here and below. However it looks like that existing ifs do not follow LLVM style. So please keep the patch as is.
https://reviews.llvm.org/D41222
More information about the llvm-commits
mailing list