Skip to content

Commit

Permalink
Require lld for hwasan tests.
Browse files Browse the repository at this point in the history
We're using relocations that are unsupported by the version of gold on the
bot, so force the use of lld. One of the tests is already using lld,
so this should be safe.

llvm-svn: 368111
  • Loading branch information
pcc committed Aug 6, 2019
1 parent 5b6785a commit feef101
Show file tree
Hide file tree
Showing 3 changed files with 5 additions and 2 deletions.
3 changes: 3 additions & 0 deletions compiler-rt/test/hwasan/CMakeLists.txt
Expand Up @@ -23,6 +23,9 @@ endforeach()
set(HWASAN_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS})
if(NOT COMPILER_RT_STANDALONE_BUILD)
list(APPEND HWASAN_TEST_DEPS hwasan)
if(COMPILER_RT_HAS_LLD)
list(APPEND HWASAN_TEST_DEPS lld)
endif()
endif()

add_lit_testsuite(check-hwasan "Running the HWAddressSanitizer tests"
Expand Down
2 changes: 1 addition & 1 deletion compiler-rt/test/hwasan/TestCases/cfi.cpp
@@ -1,4 +1,4 @@
// RUN: %clangxx_hwasan -fsanitize=cfi -fno-sanitize-trap=cfi -flto -fvisibility=hidden -fuse-ld=lld %s -o %t
// RUN: %clangxx_hwasan -fsanitize=cfi -fno-sanitize-trap=cfi -flto -fvisibility=hidden %s -o %t
// RUN: not %run %t 2>&1 | FileCheck %s

// REQUIRES: android
Expand Down
2 changes: 1 addition & 1 deletion compiler-rt/test/hwasan/lit.cfg.py
Expand Up @@ -11,7 +11,7 @@
# Setup default compiler flags used with -fsanitize=memory option.
clang_cflags = [config.target_cflags] + config.debug_info_flags
clang_cxxflags = config.cxx_mode_flags + clang_cflags
clang_hwasan_cflags = ["-fsanitize=hwaddress", "-mllvm", "-hwasan-globals"] + clang_cflags
clang_hwasan_cflags = ["-fsanitize=hwaddress", "-mllvm", "-hwasan-globals", "-fuse-ld=lld"] + clang_cflags
if config.target_arch == 'x86_64':
# This does basically the same thing as tagged-globals on aarch64. Because
# the x86_64 implementation is for testing purposes only there is no
Expand Down

0 comments on commit feef101

Please sign in to comment.