Skip to content

Commit 16ab8c0

Browse files
[compiler-rt] Default to Lit's Internal Shell
All of the tests should work with the internal shell now, at least on x86 Linux. Enable it by default for the performance/debuggability advantages. Reviewers: vitalybuka, fmayer Reviewed By: fmayer, vitalybuka Pull Request: #165148
1 parent 5ba0b91 commit 16ab8c0

File tree

1 file changed

+8
-3
lines changed

1 file changed

+8
-3
lines changed

compiler-rt/test/lit.common.cfg.py

Lines changed: 8 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -113,12 +113,17 @@ def push_dynamic_library_lookup_path(config, new_path):
113113
config.environment[dynamic_library_lookup_var] = new_ld_library_path_64
114114

115115

116+
# TODO: Consolidate the logic for turning on the internal shell by default for all LLVM test suites.
117+
# See https://github.com/llvm/llvm-project/issues/106636 for more details.
118+
#
116119
# Choose between lit's internal shell pipeline runner and a real shell. If
117120
# LIT_USE_INTERNAL_SHELL is in the environment, we use that as an override.
118-
use_lit_shell = os.environ.get("LIT_USE_INTERNAL_SHELL")
121+
use_lit_shell = True
122+
lit_shell_env = os.environ.get("LIT_USE_INTERNAL_SHELL")
123+
if lit_shell_env:
124+
use_lit_shell = lit.util.pythonize_bool(lit_shell_env)
119125
if use_lit_shell:
120-
# 0 is external, "" is default, and everything else is internal.
121-
execute_external = use_lit_shell == "0"
126+
execute_external = True
122127
else:
123128
# Otherwise we default to internal on Windows and external elsewhere, as
124129
# bash on Windows is usually very slow.

0 commit comments

Comments
 (0)