[llvm] 29cf7df - Revert "llvm-reduce: Color output of child processes"
Arthur Eubanks via llvm-commits
llvm-commits at lists.llvm.org
Mon Oct 17 14:37:56 PDT 2022
Author: Arthur Eubanks
Date: 2022-10-17T14:37:24-07:00
New Revision: 29cf7dffb81b0522231b1c154de108e059d8a4f0
URL: https://github.com/llvm/llvm-project/commit/29cf7dffb81b0522231b1c154de108e059d8a4f0
DIFF: https://github.com/llvm/llvm-project/commit/29cf7dffb81b0522231b1c154de108e059d8a4f0.diff
LOG: Revert "llvm-reduce: Color output of child processes"
This reverts commit 86e9181ded7c5b6aa67a3a84089fce850c84f27a.
Seems to be causing bot failures: https://lab.llvm.org/buildbot#builders/5/builds/28313
Plus as noted on D135632 it seems to interact badly with parallel reduction.
Added:
Modified:
llvm/tools/llvm-reduce/TestRunner.cpp
Removed:
################################################################################
diff --git a/llvm/tools/llvm-reduce/TestRunner.cpp b/llvm/tools/llvm-reduce/TestRunner.cpp
index 5681a81fd6a8..63de850de5b2 100644
--- a/llvm/tools/llvm-reduce/TestRunner.cpp
+++ b/llvm/tools/llvm-reduce/TestRunner.cpp
@@ -39,13 +39,9 @@ int TestRunner::run(StringRef Filename) {
for (int i = 0; i < 3; ++i)
Redirects.push_back(Empty);
}
-
- outs().changeColor(raw_ostream::YELLOW);
-
int Result =
sys::ExecuteAndWait(TestName, ProgramArgs, /*Env=*/None, Redirects,
/*SecondsToWait=*/0, /*MemoryLimit=*/0, &ErrMsg);
- outs().resetColor();
if (Result < 0) {
Error E = make_error<StringError>("Error running interesting-ness test: " +
More information about the llvm-commits
mailing list