diff options
author | Greg Bedwell <greg_bedwell@sn.scee.net> | 2017-08-03 17:55:54 +0000 |
---|---|---|
committer | Greg Bedwell <greg_bedwell@sn.scee.net> | 2017-08-03 17:55:54 +0000 |
commit | fd2c3795682fb9a3139f6411b1cf27c1304a66fc (patch) | |
tree | f59b2688f86fbf1dbcdcb93979248f3de8fa6d9f | |
parent | 9a18a6f08b029e452b232ba3990efb5c9e168675 (diff) | |
download | bcm5719-llvm-fd2c3795682fb9a3139f6411b1cf27c1304a66fc.tar.gz bcm5719-llvm-fd2c3795682fb9a3139f6411b1cf27c1304a66fc.zip |
Fix check-lit compatibility with multi-config CMake generators
Multi-configuration CMake generators such as those for Visual Studio or Xcode do not
specify a build config at configure time, but let the user choose at build
time. In these cases binaries go into build/${Configuration}/bin rather than
build/bin. Prior to this commit, check-lit would fail when using multi-configuration
generators as it did not know how to resolve ${Configuration} in order
to find tools such as FileCheck. This commit teaches it to resolve
llvm_tools_dir within lit using the value specified with --param
build_mode.
Differential Revision: https://reviews.llvm.org/D36263
llvm-svn: 309967
-rw-r--r-- | llvm/utils/lit/CMakeLists.txt | 2 | ||||
-rw-r--r-- | llvm/utils/lit/tests/lit.site.cfg.in | 17 |
2 files changed, 15 insertions, 4 deletions
diff --git a/llvm/utils/lit/CMakeLists.txt b/llvm/utils/lit/CMakeLists.txt index d1b91a0c8d1..d10decedaa8 100644 --- a/llvm/utils/lit/CMakeLists.txt +++ b/llvm/utils/lit/CMakeLists.txt @@ -1,7 +1,7 @@ # The configured file is not placed in the correct location # until the tests are run as we need to copy it into # a copy of the tests folder -configure_file("tests/lit.site.cfg.in" "lit.site.cfg" @ONLY) +configure_lit_site_cfg("tests/lit.site.cfg.in" "lit.site.cfg") # Lit's test suite creates output files next to the sources which makes the # source tree dirty. This is undesirable because we do out of source builds. diff --git a/llvm/utils/lit/tests/lit.site.cfg.in b/llvm/utils/lit/tests/lit.site.cfg.in index bdc1f58575e..374db49fa1e 100644 --- a/llvm/utils/lit/tests/lit.site.cfg.in +++ b/llvm/utils/lit/tests/lit.site.cfg.in @@ -1,8 +1,19 @@ -## Autogenerated by LLVM/Clang configuration. -# Do not edit! +@LIT_SITE_CFG_IN_HEADER@ + +import sys + config.llvm_src_root = "@LLVM_SOURCE_DIR@" config.llvm_obj_root = "@LLVM_BINARY_DIR@" -config.llvm_tools_dir = "@LLVM_TOOLS_BINARY_DIR@" +config.llvm_tools_dir = "@LLVM_TOOLS_DIR@" + +# Support substitution of the tools_dir with user parameters. +# This is used when we can't determine the tool dir at configuration time. +try: + config.llvm_tools_dir = config.llvm_tools_dir % lit_config.params +except KeyError: + e = sys.exc_info()[1] + key, = e.args + lit_config.fatal("unable to find %r parameter, use '--param=%s=VALUE'" % (key,key)) # Let the main config do the real work. lit_config.load_config(config, "@LLVM_BINARY_DIR@/utils/lit/tests/lit.cfg") |