Index: test/tools/llvm-readobj/error-format.test =================================================================== --- /dev/null +++ test/tools/llvm-readobj/error-format.test @@ -0,0 +1,7 @@ +## This test shows that we include the tool name in error messages. + +# RUN: not llvm-readelf %S/non-existent 2>&1 | FileCheck --check-prefix=READELF %s +# RUN: not llvm-readobj %S/non-existent 2>&1 | FileCheck --check-prefix=READOBJ %s + +# READELF: llvm-readelf{{(\.exe)?}}: error: '{{.*}}': {{[Nn]}}o such file or directory +# READOBJ: llvm-readobj{{(\.exe)?}}: error: '{{.*}}': {{[Nn]}}o such file or directory Index: tools/llvm-readobj/llvm-readobj.cpp =================================================================== --- tools/llvm-readobj/llvm-readobj.cpp +++ tools/llvm-readobj/llvm-readobj.cpp @@ -373,6 +373,8 @@ HelpResponse("\nPass @FILE as argument to read options from FILE.\n"); } // namespace opts +static StringRef ToolName; + namespace llvm { LLVM_ATTRIBUTE_NORETURN static void error(Twine Msg) { @@ -380,7 +382,7 @@ // proper place. fouts().flush(); errs() << "\n"; - WithColor::error(errs()) << Msg << "\n"; + WithColor::error(errs(), ToolName) << Msg << "\n"; exit(1); } @@ -401,11 +403,11 @@ // Flush the standard output to print the warning at a // proper place. fouts().flush(); - handleAllErrors(createFileError(Input, std::move(Err)), - [&](const ErrorInfoBase &EI) { - errs() << "\n"; - WithColor::warning(errs()) << EI.message() << "\n"; - }); + handleAllErrors( + createFileError(Input, std::move(Err)), [&](const ErrorInfoBase &EI) { + errs() << "\n"; + WithColor::warning(errs(), ToolName) << EI.message() << "\n"; + }); } LLVM_ATTRIBUTE_NORETURN void reportError(std::error_code EC, StringRef Input) { @@ -703,6 +705,7 @@ int main(int argc, const char *argv[]) { InitLLVM X(argc, argv); + ToolName = argv[0]; // Register the target printer for --version. cl::AddExtraVersionPrinter(TargetRegistry::printRegisteredTargetsForVersion);