[PATCH] D23186: [sanitizers] trace buffer API to use user-allocated buffer.
Kostya Serebryany via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 4 18:04:47 PDT 2016
kcc added inline comments.
================
Comment at: include/sanitizer/coverage_interface.h:67
@@ +66,3 @@
+ // Number of pcs recorded in the buffer.
+ // Reset by __sanitizer_reset_coverage();
+ uintptr_t __sanitizer_get_coverage_pc_buffer_pos();
----------------
Is this really what you want?
Why do you need to reset the entire coverage just to rest the pc_buffer?
Sounds like an overkill.
https://reviews.llvm.org/D23186
More information about the llvm-commits
mailing list