[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