[PATCH] D14572: sanitizer: add support for trace coverage
Kostya Serebryany via llvm-commits
llvm-commits at lists.llvm.org
Wed Nov 11 09:56:43 PST 2015
kcc accepted this revision.
kcc added a comment.
This revision is now accepted and ready to land.
LGTM with two nits.
Thanks!
================
Comment at: include/sanitizer/coverage_interface.h:69
@@ +68,3 @@
+ // out of memory.
+ bool __sanitizer_trace_enable(uintptr_t size);
+ // Disable tracing for the current thread and free all associated resources.
----------------
here and below use int instead of bool.
This API can theoretically be used in C
================
Comment at: lib/Transforms/Instrumentation/SanitizerCoverage.cpp:80
@@ +79,3 @@
+ ClTracing("sanitizer-coverage-trace",
+ cl::desc("Tacing: insert callbacks at every basic"
+ " function/block/edge"),
----------------
typo: Tacing
http://reviews.llvm.org/D14572
More information about the llvm-commits
mailing list