[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