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