From 0a845582010ea6e4664c8d9fc7e676b0b8612596 Mon Sep 17 00:00:00 2001 From: "Joel E. Denny" Date: Thu, 1 Aug 2019 22:26:51 +0000 Subject: [PATCH] Revert r366980: "[lit] Protect full test suite from FILECHECK_OPTS" Windows bots are broken. See recent D65335 and D65156 comments. git-svn-id: https://llvm.org/svn/llvm-project/llvm/trunk@367627 91177308-0d34-0410-b5e6-96231b3b80d8 --- utils/lit/tests/lit.cfg | 11 ++--------- utils/lit/tests/shtest-run-at-line.py | 2 +- 2 files changed, 3 insertions(+), 10 deletions(-) diff --git a/utils/lit/tests/lit.cfg b/utils/lit/tests/lit.cfg index 2830956f80d..ebdcb5000b9 100644 --- a/utils/lit/tests/lit.cfg +++ b/utils/lit/tests/lit.cfg @@ -44,17 +44,10 @@ for attribute in ('llvm_tools_dir', 'lit_tools_dir'): if directory: llvm_config.with_environment('PATH', directory, append_path=True) -# This test suite calls %{lit} to test lit's behavior for the sample test -# suites in %{inputs}. This test suite's results are then determined in part -# by %{lit}'s textual output, which includes the output of FileCheck calls -# within %{inputs}'s test suites. Thus, %{lit} clears environment variables -# that can affect FileCheck's output. config.substitutions.append(('%{inputs}', os.path.join( config.test_source_root, 'Inputs'))) -config.substitutions.append(('%{lit}', - "{env} %{{python}} {lit}".format( - env="env -u FILECHECK_OPTS -u FILECHECK_DUMP_INPUT_ON_FAILURE", - lit=os.path.join(lit_path, 'lit.py')))) +config.substitutions.append(('%{lit}', "%%{python} %s" % ( + os.path.join(lit_path, 'lit.py'),))) config.substitutions.append(('%{python}', '"%s"' % (sys.executable))) # Enable coverage.py reporting, assuming the coverage module has been installed diff --git a/utils/lit/tests/shtest-run-at-line.py b/utils/lit/tests/shtest-run-at-line.py index cd0e08137ee..7e5d53b8e26 100644 --- a/utils/lit/tests/shtest-run-at-line.py +++ b/utils/lit/tests/shtest-run-at-line.py @@ -1,7 +1,7 @@ # Check that -vv makes the line number of the failing RUN command clear. # (-v is actually sufficient in the case of the internal shell.) # -# RUN: not %{lit} -j 1 -vv %{inputs}/shtest-run-at-line > %t.out +# RUN: env -u FILECHECK_OPTS not %{lit} -j 1 -vv %{inputs}/shtest-run-at-line > %t.out # RUN: FileCheck --input-file %t.out %s # # END. -- 2.40.0