summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorGreg Bedwell <greg_bedwell@sn.scee.net>2017-08-03 17:55:54 +0000
committerGreg Bedwell <greg_bedwell@sn.scee.net>2017-08-03 17:55:54 +0000
commitfd2c3795682fb9a3139f6411b1cf27c1304a66fc (patch)
treef59b2688f86fbf1dbcdcb93979248f3de8fa6d9f
parent9a18a6f08b029e452b232ba3990efb5c9e168675 (diff)
downloadbcm5719-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.txt2
-rw-r--r--llvm/utils/lit/tests/lit.site.cfg.in17
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")
OpenPOWER on IntegriCloud