Commit
This commit does not belong to any branch on this repository, and may belong to a fork outside of the repository.
Revert "Make -fsanitize=scudo use scudo_standalone. Delete check-scudo."
It broke the build, see comments on code review. > Leaves the implementation and tests files in-place for right now, but > deletes the ability to build the old sanitizer-common based scudo. This > has been on life-support for a long time, and the newer scudo_standalone > is much better supported and maintained. > > Also patches up some GWP-ASan wording, primarily related to the fact > that -fsanitize=scudo now is scudo_standalone, and therefore the way to > reference the GWP-ASan options through the environment variable has > changed. > > Future follow-up patches will delete the original scudo, and migrate all > its tests over to be part of the scudo_standalone test suite. > > Reviewed By: vitalybuka > > Differential Revision: https://reviews.llvm.org/D138157 This reverts commit ab1a599.
- Loading branch information
Showing
8 changed files
with
87 additions
and
27 deletions.
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,35 @@ | ||
set(SCUDO_LIT_SOURCE_DIR ${CMAKE_CURRENT_SOURCE_DIR}) | ||
set(SCUDO_LIT_BINARY_DIR ${CMAKE_CURRENT_BINARY_DIR}) | ||
|
||
set(SCUDO_TESTSUITES) | ||
|
||
set(SCUDO_TEST_DEPS ${SANITIZER_COMMON_LIT_TEST_DEPS}) | ||
if(NOT COMPILER_RT_STANDALONE_BUILD) | ||
list(APPEND SCUDO_TEST_DEPS scudo) | ||
endif() | ||
|
||
configure_lit_site_cfg( | ||
${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.py.in | ||
${CMAKE_CURRENT_BINARY_DIR}/lit.site.cfg.py | ||
) | ||
|
||
set(SCUDO_TEST_ARCH ${SCUDO_SUPPORTED_ARCH}) | ||
foreach(arch ${SCUDO_TEST_ARCH}) | ||
set(SCUDO_TEST_TARGET_ARCH ${arch}) | ||
string(TOLOWER "-${arch}" SCUDO_TEST_CONFIG_SUFFIX) | ||
get_test_cc_for_arch(${arch} SCUDO_TEST_TARGET_CC SCUDO_TEST_TARGET_CFLAGS) | ||
string(TOUPPER ${arch} ARCH_UPPER_CASE) | ||
set(CONFIG_NAME ${ARCH_UPPER_CASE}${OS_NAME}Config) | ||
|
||
configure_lit_site_cfg( | ||
${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.py.in | ||
${CMAKE_CURRENT_BINARY_DIR}/${CONFIG_NAME}/lit.site.cfg.py) | ||
list(APPEND SCUDO_TESTSUITES ${CMAKE_CURRENT_BINARY_DIR}/${CONFIG_NAME}) | ||
endforeach() | ||
|
||
add_subdirectory(standalone) | ||
|
||
add_lit_testsuite(check-scudo "Running the Scudo Hardened Allocator tests" | ||
${SCUDO_TESTSUITES} | ||
DEPENDS ${SCUDO_TEST_DEPS}) | ||
set_target_properties(check-scudo PROPERTIES FOLDER "Compiler-RT Misc") |
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters