From b166d7e72512d699fadd2a049dac2ba6a7af888e Mon Sep 17 00:00:00 2001 From: Nico Weber Date: Sat, 19 Jan 2019 00:09:43 +0000 Subject: [PATCH] Use llvm_canonicalize_cmake_booleans for LLVM_LIBXML2_ENABLED [lld] r291284 added a nice mechanism to consistently pass CMake on/off toggles to lit. This change uses it for LLVM_LIBXML2_ENABLED too (which was added around the same time and doesn't use the new system yet). No intended behavior change. Differential Revision: https://reviews.llvm.org/D56912 llvm-svn: 351614 --- lld/test/CMakeLists.txt | 4 +++- lld/test/lit.cfg.py | 6 +++--- lld/test/lit.site.cfg.py.in | 2 +- 3 files changed, 7 insertions(+), 5 deletions(-) diff --git a/lld/test/CMakeLists.txt b/lld/test/CMakeLists.txt index 1b908335c4a1..64aa1f7cb3e2 100644 --- a/lld/test/CMakeLists.txt +++ b/lld/test/CMakeLists.txt @@ -14,7 +14,9 @@ if(LLD_BUILT_STANDALONE) endif() llvm_canonicalize_cmake_booleans( - HAVE_LIBZ) + HAVE_LIBZ + LLVM_LIBXML2_ENABLED + ) configure_lit_site_cfg( ${CMAKE_CURRENT_SOURCE_DIR}/lit.site.cfg.py.in diff --git a/lld/test/lit.cfg.py b/lld/test/lit.cfg.py index e53ff8740035..350b40a10081 100644 --- a/lld/test/lit.cfg.py +++ b/lld/test/lit.cfg.py @@ -87,11 +87,11 @@ config.environment['LLD_IN_TEST'] = '1' # Indirectly check if the mt.exe Microsoft utility exists by searching for # cvtres, which always accompanies it. Alternatively, check if we can use # libxml2 to merge manifests. -if (lit.util.which('cvtres', config.environment['PATH'])) or \ - (config.llvm_libxml2_enabled == '1'): +if (lit.util.which('cvtres', config.environment['PATH']) or + config.llvm_libxml2_enabled): config.available_features.add('manifest_tool') -if (config.llvm_libxml2_enabled == '1'): +if config.llvm_libxml2_enabled: config.available_features.add('libxml2') if config.have_dia_sdk: diff --git a/lld/test/lit.site.cfg.py.in b/lld/test/lit.site.cfg.py.in index 7475ac7eaa7f..07ffc2303e11 100644 --- a/lld/test/lit.site.cfg.py.in +++ b/lld/test/lit.site.cfg.py.in @@ -7,7 +7,7 @@ config.llvm_src_root = "@LLVM_SOURCE_DIR@" config.llvm_obj_root = "@LLVM_BINARY_DIR@" config.llvm_tools_dir = "@LLVM_TOOLS_DIR@" config.llvm_libs_dir = "@LLVM_LIBS_DIR@" -config.llvm_libxml2_enabled = "@LLVM_LIBXML2_ENABLED@" +config.llvm_libxml2_enabled = @LLVM_LIBXML2_ENABLED@ config.lit_tools_dir = "@LLVM_LIT_TOOLS_DIR@" config.lld_obj_root = "@LLD_BINARY_DIR@" config.lld_libs_dir = "@LLVM_LIBRARY_OUTPUT_INTDIR@"