[PATCH] D32194: Turn symbolication on for ASan unit test.

[Aleksey Shlyapnikov via Phabricator via llvm-commits]

This revision was automatically updated to reflect the committed changes.
Closed by commit rL300748: Turn symbolization on for ASan unit test. (authored by alekseyshl).

Changed prior to commit:
  https://reviews.llvm.org/D32194?vs=95799&id=95812#toc

Repository:
  rL LLVM

https://reviews.llvm.org/D32194

Files:
  compiler-rt/trunk/lib/asan/tests/asan_test_main.cc


Index: compiler-rt/trunk/lib/asan/tests/asan_test_main.cc
===================================================================
--- compiler-rt/trunk/lib/asan/tests/asan_test_main.cc
+++ compiler-rt/trunk/lib/asan/tests/asan_test_main.cc
@@ -13,15 +13,26 @@
 #include "asan_test_utils.h"
 #include "sanitizer_common/sanitizer_platform.h"
 
-// Default ASAN_OPTIONS for the unit tests. Let's turn symbolication off to
-// speed up testing (unit tests don't use it anyway).
+// Default ASAN_OPTIONS for the unit tests.
 extern "C" const char* __asan_default_options() {
 #if SANITIZER_MAC
   // On Darwin, we default to `abort_on_error=1`, which would make tests run
-  // much slower. Let's override this and run lit tests with 'abort_on_error=0'.
-  // Also, make sure we do not overwhelm the syslog while testing.
+  // much slower. Let's override this and run lit tests with 'abort_on_error=0'
+  // and make sure we do not overwhelm the syslog while testing. Also, let's
+  // turn symbolization off to speed up testing, especially when not running
+  // with llvm-symbolizer but with atos.
   return "symbolize=false:abort_on_error=0:log_to_syslog=0";
+#elif SANITIZER_PPC || defined(__thumb__)
+  // On PowerPC and ARM Thumb, a couple tests involving pthread_exit fail due to
+  // leaks detected by LSan. pthread_exit tries to perform unwinding that leads
+  // to dlopen'ing libgcc_s.so. dlopen mallocs "libgcc_s.so" string which
+  // confuses LSan, it fails to realize that this allocation happens in dynamic
+  // linker and should be ignored.  Symbolized leak report is required to define
+  // a suppression for this known problem.
+  return "";
 #else
+  // Let's turn symbolization off to speed up testing (more than 3 times speedup
+  // observed).
   return "symbolize=false";
 #endif
 }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D32194.95812.patch
Type: text/x-patch
Size: 1802 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170419/4a54dfc5/attachment.bin>