[compiler-rt] r278308 - [asan] abort_on_error=1 by default on Android.
Evgeniy Stepanov via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 10 17:26:29 PDT 2016
Author: eugenis
Date: Wed Aug 10 19:26:29 2016
New Revision: 278308
URL: http://llvm.org/viewvc/llvm-project?rev=278308&view=rev
Log:
[asan] abort_on_error=1 by default on Android.
With this change, the default behavior on error is to call abort()
instead of _exit(). This should help the OS to capture a tombstone of
the error.
RAM usage of the lit test suite goes up because of all the tombstone
gathering, so I'm limiting the parallelism of the test target.
Previously it was based on the number of the CPUs on the host
machine, which is definitely wrong.
Modified:
compiler-rt/trunk/lib/sanitizer_common/sanitizer_flags.inc
compiler-rt/trunk/test/asan/CMakeLists.txt
Modified: compiler-rt/trunk/lib/sanitizer_common/sanitizer_flags.inc
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/lib/sanitizer_common/sanitizer_flags.inc?rev=278308&r1=278307&r2=278308&view=diff
==============================================================================
--- compiler-rt/trunk/lib/sanitizer_common/sanitizer_flags.inc (original)
+++ compiler-rt/trunk/lib/sanitizer_common/sanitizer_flags.inc Wed Aug 10 19:26:29 2016
@@ -213,7 +213,7 @@ COMMON_FLAG(bool, decorate_proc_maps, fa
COMMON_FLAG(int, exitcode, 1, "Override the program exit status if the tool "
"found an error")
COMMON_FLAG(
- bool, abort_on_error, SANITIZER_MAC,
+ bool, abort_on_error, SANITIZER_ANDROID || SANITIZER_MAC,
"If set, the tool calls abort() instead of _exit() after printing the "
"error report.")
COMMON_FLAG(bool, suppress_equal_pcs, true,
Modified: compiler-rt/trunk/test/asan/CMakeLists.txt
URL: http://llvm.org/viewvc/llvm-project/compiler-rt/trunk/test/asan/CMakeLists.txt?rev=278308&r1=278307&r2=278308&view=diff
==============================================================================
--- compiler-rt/trunk/test/asan/CMakeLists.txt (original)
+++ compiler-rt/trunk/test/asan/CMakeLists.txt Wed Aug 10 19:26:29 2016
@@ -100,9 +100,15 @@ if(COMPILER_RT_INCLUDE_TESTS)
endif()
endif()
+set(LIT_ARGS)
+if(ANDROID)
+ set(LIT_ARGS -j5)
+endif()
+
add_lit_testsuite(check-asan "Running the AddressSanitizer tests"
${ASAN_TESTSUITES}
- DEPENDS ${ASAN_TEST_DEPS})
+ DEPENDS ${ASAN_TEST_DEPS}
+ ARGS ${LIT_ARGS})
set_target_properties(check-asan PROPERTIES FOLDER "Compiler-RT Misc")
if(COMPILER_RT_ASAN_HAS_STATIC_RUNTIME)
More information about the llvm-commits
mailing list