[llvm] 5125f47 - Revert "Reapply "[llvm] Use lit internal shell by default""
Aiden Grossman via llvm-commits
llvm-commits at lists.llvm.org
Thu Sep 11 08:58:09 PDT 2025
Author: Aiden Grossman
Date: 2025-09-11T15:57:32Z
New Revision: 5125f476b2f90ccf157c78d73bc6fe14c4413a27
URL: https://github.com/llvm/llvm-project/commit/5125f476b2f90ccf157c78d73bc6fe14c4413a27
DIFF: https://github.com/llvm/llvm-project/commit/5125f476b2f90ccf157c78d73bc6fe14c4413a27.diff
LOG: Revert "Reapply "[llvm] Use lit internal shell by default""
This reverts commit f869d7a1b100c9f9fc5bb06effcf4f73346c0919.
This broke some debuginfod tests that are not run by default in most CI
configurations. Reverting for now until I have time to update the tests
and figure out how to get them built locally.
Added:
Modified:
llvm/test/lit.cfg.py
Removed:
################################################################################
diff --git a/llvm/test/lit.cfg.py b/llvm/test/lit.cfg.py
index e8861e29be707..867a44be56727 100644
--- a/llvm/test/lit.cfg.py
+++ b/llvm/test/lit.cfg.py
@@ -17,17 +17,6 @@
# name: The name of this test suite.
config.name = "LLVM"
-# TODO: Consolidate the logic for turning on the internal shell by default for all LLVM test suites.
-# See https://github.com/llvm/llvm-project/issues/106636 for more details.
-#
-# We prefer the lit internal shell which provides a better user experience on failures
-# and is faster unless the user explicitly disables it with LIT_USE_INTERNAL_SHELL=0
-# env var.
-use_lit_shell = True
-lit_shell_env = os.environ.get("LIT_USE_INTERNAL_SHELL")
-if lit_shell_env:
- use_lit_shell = lit.util.pythonize_bool(lit_shell_env)
-
# testFormat: The test format to use to interpret tests.
extra_substitutions = extra_substitutions = (
[
@@ -37,7 +26,9 @@
if config.enable_profcheck
else []
)
-config.test_format = lit.formats.ShTest(not use_lit_shell, extra_substitutions)
+config.test_format = lit.formats.ShTest(
+ not llvm_config.use_lit_shell, extra_substitutions
+)
# suffixes: A list of file extensions to treat as test files. This is overriden
# by individual lit.local.cfg files in the test subdirectories.
More information about the llvm-commits
mailing list