Skip to content

Commit 75e7b6e

Browse files
committed
[Sanitizer][Apple] Enable sanitizer common unittests for arm64 archs on Apple
This patch enables sanitizer common unit tests for arm64 architecture only on apple devices. It also lowers the expected compression ratio for 64 bit machines. Unsure of justification of 130. On apple arm64 we're seeing this number comeout to 128.6 rdar://101436019 Differential Revision: https://reviews.llvm.org/D141170
1 parent 8cdb4f9 commit 75e7b6e

File tree

2 files changed

+2
-1
lines changed

2 files changed

+2
-1
lines changed

compiler-rt/lib/sanitizer_common/tests/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -5,6 +5,7 @@ clang_compiler_add_cxx_check()
55
# FIXME: use SANITIZER_COMMON_SUPPORTED_ARCH here
66
filter_available_targets(SANITIZER_UNITTEST_SUPPORTED_ARCH x86_64 i386 mips64 mips64el riscv64 sparcv9 sparc)
77
if(APPLE)
8+
list(APPEND SANITIZER_UNITTEST_SUPPORTED_ARCH arm64)
89
darwin_filter_host_archs(SANITIZER_UNITTEST_SUPPORTED_ARCH SANITIZER_UNITTEST_SUPPORTED_ARCH)
910
endif()
1011

compiler-rt/lib/sanitizer_common/tests/sanitizer_stack_store_test.cpp

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -138,7 +138,7 @@ INSTANTIATE_TEST_SUITE_P(
138138
StackStorePackTest::ParamType(StackStore::Compression::Delta,
139139
FIRST_32_SECOND_64(2, 6)),
140140
StackStorePackTest::ParamType(StackStore::Compression::LZW,
141-
FIRST_32_SECOND_64(60, 130)),
141+
FIRST_32_SECOND_64(60, 125)),
142142
}));
143143

144144
TEST_P(StackStorePackTest, PackUnpack) {

0 commit comments

Comments
 (0)