Index: llvm/trunk/lib/ProfileData/InstrProf.cpp =================================================================== --- llvm/trunk/lib/ProfileData/InstrProf.cpp +++ llvm/trunk/lib/ProfileData/InstrProf.cpp @@ -51,13 +51,13 @@ case instrprof_error::unknown_function: return "No profile data available for function"; case instrprof_error::hash_mismatch: - return "Function hash mismatch"; + return "Function control flow change detected (hash mismatch)"; case instrprof_error::count_mismatch: - return "Function count mismatch"; + return "Function basic block count change detected (counter mismatch)"; case instrprof_error::counter_overflow: return "Counter overflow"; case instrprof_error::value_site_count_mismatch: - return "Function's value site counts mismatch"; + return "Function value site count change detected (counter mismatch)"; } llvm_unreachable("A value of instrprof_error has no message."); } Index: llvm/trunk/test/tools/llvm-profdata/count-mismatch.proftext =================================================================== --- llvm/trunk/test/tools/llvm-profdata/count-mismatch.proftext +++ llvm/trunk/test/tools/llvm-profdata/count-mismatch.proftext @@ -14,7 +14,8 @@ # The hash matches, but we can't combine these because the number of # counters differs. -# MERGE_ERRS: count-mismatch.proftext: foo: Function count mismatch +# MERGE_ERRS: count-mismatch.proftext: foo: Function basic block count change detected (counter mismatch) +# MERGE_ERRS: Make sure that all profile data to be merged is generated from the same binary. foo 1024 3 Index: llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp =================================================================== --- llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp +++ llvm/trunk/tools/llvm-profdata/llvm-profdata.cpp @@ -27,6 +27,8 @@ #include "llvm/Support/Signals.h" #include "llvm/Support/raw_ostream.h" +#include + using namespace llvm; static void exitWithError(const Twine &Message, @@ -41,7 +43,8 @@ ::exit(1); } -static void exitWithErrorCode(const std::error_code &Error, StringRef Whence = "") { +static void exitWithErrorCode(const std::error_code &Error, + StringRef Whence = "") { if (Error.category() == instrprof_category()) { instrprof_error instrError = static_cast(Error.value()); if (instrError == instrprof_error::unrecognized_format) { @@ -57,6 +60,32 @@ enum ProfileKinds { instr, sample }; } +static void handleMergeWriterError(std::error_code &Error, + StringRef WhenceFile = "", + StringRef WhenceFunction = "", + bool ShowHint = true) +{ + if (!WhenceFile.empty()) + errs() << WhenceFile << ": "; + if (!WhenceFunction.empty()) + errs() << WhenceFunction << ": "; + errs() << Error.message() << "\n"; + + if (ShowHint) { + StringRef Hint = ""; + if (Error.category() == instrprof_category()) { + instrprof_error instrError = static_cast(Error.value()); + if (instrError == instrprof_error::count_mismatch) { + Hint = "Make sure that all profile data to be merged is generated " \ + "from the same binary."; + } + } + + if (!Hint.empty()) + errs() << Hint << "\n"; + } +} + static void mergeInstrProfile(const cl::list &Inputs, StringRef OutputFilename) { if (OutputFilename.compare("-") == 0) @@ -68,15 +97,20 @@ exitWithErrorCode(EC, OutputFilename); InstrProfWriter Writer; + std::set WriterErrorCodes; for (const auto &Filename : Inputs) { auto ReaderOrErr = InstrProfReader::create(Filename); if (std::error_code ec = ReaderOrErr.getError()) exitWithErrorCode(ec, Filename); auto Reader = std::move(ReaderOrErr.get()); - for (auto &I : *Reader) - if (std::error_code EC = Writer.addRecord(std::move(I))) - errs() << Filename << ": " << I.Name << ": " << EC.message() << "\n"; + for (auto &I : *Reader) { + if (std::error_code EC = Writer.addRecord(std::move(I))) { + // Only show hint the first time an error occurs. + bool firstTime = WriterErrorCodes.insert(EC).second; + handleMergeWriterError(EC, Filename, I.Name, firstTime); + } + } if (Reader->hasError()) exitWithErrorCode(Reader->getError(), Filename); }