Revert "Reapply "[compiler-rt] Default to Lit's Internal Shell" (#168232)"
This reverts commit bde9062418.
This caused failures on Darwin that were not caught by upstream
buildbots. Reverting for now to give myself some time to fix.
This commit is contained in:
@@ -113,9 +113,6 @@ def push_dynamic_library_lookup_path(config, new_path):
|
||||
config.environment[dynamic_library_lookup_var] = new_ld_library_path_64
|
||||
|
||||
|
||||
# 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.
|
||||
#
|
||||
# Choose between lit's internal shell pipeline runner and a real shell. If
|
||||
# LIT_USE_INTERNAL_SHELL is in the environment, we use that as an override.
|
||||
use_lit_shell = os.environ.get("LIT_USE_INTERNAL_SHELL")
|
||||
@@ -123,8 +120,9 @@ if use_lit_shell:
|
||||
# 0 is external, "" is default, and everything else is internal.
|
||||
execute_external = use_lit_shell == "0"
|
||||
else:
|
||||
# Otherwise we default to internal everywhere.
|
||||
execute_external = False
|
||||
# Otherwise we default to internal on Windows and external elsewhere, as
|
||||
# bash on Windows is usually very slow.
|
||||
execute_external = not sys.platform in ["win32"]
|
||||
|
||||
# Allow expanding substitutions that are based on other substitutions
|
||||
config.recursiveExpansionLimit = 10
|
||||
|
||||
Reference in New Issue
Block a user