[PATCH] Tests and interface for LeakSanitizer.

Alexander Potapenko glider at google.com
Wed Apr 17 03:28:28 PDT 2013



================
Comment at: lib/lsan/lsan_interface.h:38
@@ +37,3 @@
+void __lsan_enable_all_sources();
+void __lsan_enable_source(enum __lsan_sources source);
+void __lsan_disable_source(enum __lsan_sources source);
----------------
Sergey Matveev wrote:
> Alexander Potapenko wrote:
> > I'm curious if __lsan_enable_xxx takes any effect after __lsan_start_local_leak_checking() is called.
> Actually, it only affects the behavior of __lsan_report_local_leaks (and __lsan_do_global_leak_check). Maybe those flags should go into a parameter instead -- but then you would have to spell out every flag when calling those functions in the typical case. I haven't been able to come up with a better alternative so far.
This is fine, but the API state machine needs some clarification in the comments.


http://llvm-reviews.chandlerc.com/D677



More information about the llvm-commits mailing list