[PATCH] Tests and interface for LeakSanitizer.
    Sergey Matveev 
    earthdok at google.com
       
    Mon Apr 22 08:12:52 PDT 2013
    
    
  
================
Comment at: lib/lsan/tests/lsan_test.cc:165
@@ +164,3 @@
+  // To prevent the compiler from optimizing out this function.
+  (void) __sync_fetch_and_xor(&locals[0], p);
+  // Hide the result, just to suppress the compiler warning.
----------------
Alexander Potapenko wrote:
> Isn't volatile enough to prevent the optimization?
Not in my experience.
================
Comment at: lib/lsan/lsan_testing_interface.h:19
@@ +18,3 @@
+#include "sanitizer_common/sanitizer_internal_defs.h"
+#include "sanitizer_common/sanitizer_common.h"
+
----------------
Alexander Potapenko wrote:
> Nit: please fix the order of includes.
Ok.
http://llvm-reviews.chandlerc.com/D677
BRANCH
  lsan_interface_and_tests
ARCANIST PROJECT
  compiler-rt
    
    
More information about the llvm-commits
mailing list