[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