[PATCH] D123967: Disable update_cc_test_checks.py tests in stand-alone builds

Tom Stellard via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 18 16:51:58 PDT 2022


tstellar created this revision.
Herald added a subscriber: mgorny.
Herald added a project: All.
tstellar requested review of this revision.
Herald added a project: clang.

The script is located in the llvm/ sub-directory, so it is not available
for when doing a stand-alone build.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D123967

Files:
  clang/test/CMakeLists.txt
  clang/test/lit.site.cfg.py.in
  clang/test/utils/update_cc_test_checks/lit.local.cfg


Index: clang/test/utils/update_cc_test_checks/lit.local.cfg
===================================================================
--- clang/test/utils/update_cc_test_checks/lit.local.cfg
+++ clang/test/utils/update_cc_test_checks/lit.local.cfg
@@ -10,27 +10,36 @@
     from pipes import quote as shell_quote
 
 
-config.test_format = lit.formats.ShTest(execute_external=False)
-config.suffixes = ['.test']
-
-clang_path = os.path.join(config.clang_tools_dir, 'clang')
-extra_args = '--clang ' + shell_quote(clang_path)
-opt_path = os.path.join(config.llvm_tools_dir, 'opt')
-extra_args += ' --opt ' + shell_quote(opt_path)
-script_path = os.path.join(config.llvm_src_root, 'utils',
-                           'update_cc_test_checks.py')
-assert os.path.isfile(script_path)
-# Windows: llvm-lit.py, Linux: llvm-lit
-if config.llvm_external_lit:
-    lit = config.llvm_external_lit
+if config.stand_alone_build:
+    # These tests require the update_cc_test_checks.py script from the llvm
+    # source tree, so skip these tests if we are doing stand-alone builds.
+    # These tests are only relevant to developers working with the
+    # update_cc_test_checks.py tool they don't don't provide any coverage
+    # for any of the clang source code.
+    config.unsupported = True
 else:
-    lit = shell_quote(glob.glob(os.path.join(config.llvm_tools_dir, 'llvm-lit*'))[0])
-python = shell_quote(config.python_executable)
-config.substitutions.append(
-    ('%update_cc_test_checks', "%s %s %s" % (
-        python, shell_quote(script_path), extra_args)))
-config.substitutions.append(
-    ('%clang_tools_dir', shell_quote(config.clang_tools_dir)))
-config.substitutions.append(
-    ('%lit', "%s %s -Dclang_lit_site_cfg=%s -j1 -vv" % (
-        python, lit, shell_quote(config.clang_lit_site_cfg))))
+
+    config.test_format = lit.formats.ShTest(execute_external=False)
+    config.suffixes = ['.test']
+
+    clang_path = os.path.join(config.clang_tools_dir, 'clang')
+    extra_args = '--clang ' + shell_quote(clang_path)
+    opt_path = os.path.join(config.llvm_tools_dir, 'opt')
+    extra_args += ' --opt ' + shell_quote(opt_path)
+    script_path = os.path.join(config.llvm_src_root, 'utils',
+                               'update_cc_test_checks.py')
+    assert os.path.isfile(script_path)
+    # Windows: llvm-lit.py, Linux: llvm-lit
+    if config.llvm_external_lit:
+        lit = config.llvm_external_lit
+    else:
+        lit = shell_quote(glob.glob(os.path.join(config.llvm_tools_dir, 'llvm-lit*'))[0])
+    python = shell_quote(config.python_executable)
+    config.substitutions.append(
+        ('%update_cc_test_checks', "%s %s %s" % (
+            python, shell_quote(script_path), extra_args)))
+    config.substitutions.append(
+        ('%clang_tools_dir', shell_quote(config.clang_tools_dir)))
+    config.substitutions.append(
+        ('%lit', "%s %s -Dclang_lit_site_cfg=%s -j1 -vv" % (
+            python, lit, shell_quote(config.clang_lit_site_cfg))))
Index: clang/test/lit.site.cfg.py.in
===================================================================
--- clang/test/lit.site.cfg.py.in
+++ clang/test/lit.site.cfg.py.in
@@ -37,6 +37,7 @@
 config.has_plugins = @CLANG_PLUGIN_SUPPORT@
 config.clang_vendor_uti = "@CLANG_VENDOR_UTI@"
 config.llvm_external_lit = path(r"@LLVM_EXTERNAL_LIT@")
+config.stand_alone_build = @CLANG_BUILT_STANDALONE@
 
 import lit.llvm
 lit.llvm.initialize(lit_config, config)
Index: clang/test/CMakeLists.txt
===================================================================
--- clang/test/CMakeLists.txt
+++ clang/test/CMakeLists.txt
@@ -14,6 +14,7 @@
   LLVM_ENABLE_PER_TARGET_RUNTIME_DIR
   LLVM_ENABLE_THREADS
   LLVM_WITH_Z3
+  CLANG_BUILT_STANDALONE
   )
 
 configure_lit_site_cfg(


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123967.423488.patch
Type: text/x-patch
Size: 3752 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220418/95a64dd8/attachment-0001.bin>


More information about the cfe-commits mailing list