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