kcc added a comment.
Did you check this on something other than the unit tests?
E.g. a couple of benchmarks from fuzzer-test-suite?
================
Comment at: llvm/lib/Transforms/Instrumentation/SanitizerCoverage.cpp:177
+bool IsLeafFunc(const Function &F) {
+ for (const BasicBlock &BB : F.getBasicBlockList())
----------------
we already have a linear scan in SanitizerCoverageModule::runOnFunction --
don't introduce a second one.
Also, there is InvokeInst in addition to CallInst, see the loop in
runOnFunction.
You can simply extend the loop in runOnFunction to set a flag if the function
has non-intrin calls/ invokes
https://reviews.llvm.org/D37156
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
