r208988 - Tooling: use stderr, not stdout for missing command line messages

Alp Toker alp at nuanti.com
Fri May 16 06:45:29 PDT 2014


Author: alp
Date: Fri May 16 08:45:29 2014
New Revision: 208988

URL: http://llvm.org/viewvc/llvm-project?rev=208988&view=rev
Log:
Tooling: use stderr, not stdout for missing command line messages

Matches other tooling output that use stderr.

Modified:
    cfe/trunk/lib/Tooling/Tooling.cpp

Modified: cfe/trunk/lib/Tooling/Tooling.cpp
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/Tooling/Tooling.cpp?rev=208988&r1=208987&r2=208988&view=diff
==============================================================================
--- cfe/trunk/lib/Tooling/Tooling.cpp (original)
+++ cfe/trunk/lib/Tooling/Tooling.cpp Fri May 16 08:45:29 2014
@@ -256,7 +256,7 @@ bool FrontendActionFactory::runInvocatio
   // pass it to an std::unique_ptr declared after the Compiler variable.
   std::unique_ptr<FrontendAction> ScopedToolAction(create());
 
-  // Create the compilers actual diagnostics engine.
+  // Create the compiler's actual diagnostics engine.
   Compiler.createDiagnostics(DiagConsumer, /*ShouldOwnClient=*/false);
   if (!Compiler.hasDiagnostics())
     return false;
@@ -290,7 +290,7 @@ ClangTool::ClangTool(const CompilationDa
       // about the .cc files that were not found, and the use case where I
       // specify all files I want to run over explicitly, where this should
       // be an error. We'll want to add an option for this.
-      llvm::outs() << "Skipping " << File << ". Command line not found.\n";
+      llvm::errs() << "Skipping " << File << ". Compile command not found.\n";
     }
   }
 }





More information about the cfe-commits mailing list