diff options
| author | Justin Bogner <mail@justinbogner.com> | 2014-03-21 17:46:22 +0000 |
|---|---|---|
| committer | Justin Bogner <mail@justinbogner.com> | 2014-03-21 17:46:22 +0000 |
| commit | b9bd7f85a76263e6092198435b59b85a1b65aa0f (patch) | |
| tree | 70639306072c6328a0bfdd28606e0e30684a7f22 /llvm/lib | |
| parent | c07cc8f370704dc28db6d8e75ab9c5a9dfbca9c2 (diff) | |
| download | bcm5719-llvm-b9bd7f85a76263e6092198435b59b85a1b65aa0f.tar.gz bcm5719-llvm-b9bd7f85a76263e6092198435b59b85a1b65aa0f.zip | |
ProfileData: Introduce InstrProfWriter using the naive text format
This isn't a format we'll want to write out in practice, but moving it
to the writer library simplifies llvm-profdata and isolates it from
further changes to the format.
This also allows us to update the tests to not rely on the text output
format.
llvm-svn: 204489
Diffstat (limited to 'llvm/lib')
| -rw-r--r-- | llvm/lib/ProfileData/CMakeLists.txt | 1 | ||||
| -rw-r--r-- | llvm/lib/ProfileData/InstrProf.cpp | 6 | ||||
| -rw-r--r-- | llvm/lib/ProfileData/InstrProfWriter.cpp | 58 |
3 files changed, 65 insertions, 0 deletions
diff --git a/llvm/lib/ProfileData/CMakeLists.txt b/llvm/lib/ProfileData/CMakeLists.txt index cef3a5336bd..5204afcdc4e 100644 --- a/llvm/lib/ProfileData/CMakeLists.txt +++ b/llvm/lib/ProfileData/CMakeLists.txt @@ -1,6 +1,7 @@ add_llvm_library(LLVMProfileData InstrProf.cpp InstrProfReader.cpp + InstrProfWriter.cpp LINK_LIBS LLVMSupport diff --git a/llvm/lib/ProfileData/InstrProf.cpp b/llvm/lib/ProfileData/InstrProf.cpp index 329abf84e6e..ecabdd75bdf 100644 --- a/llvm/lib/ProfileData/InstrProf.cpp +++ b/llvm/lib/ProfileData/InstrProf.cpp @@ -39,6 +39,12 @@ class InstrProfErrorCategoryType : public error_category { return "Malformed profile data"; case instrprof_error::unknown_function: return "No profile data available for function"; + case instrprof_error::hash_mismatch: + return "Function hash mismatch"; + case instrprof_error::count_mismatch: + return "Function count mismatch"; + case instrprof_error::counter_overflow: + return "Counter overflow"; } llvm_unreachable("A value of instrprof_error has no message."); } diff --git a/llvm/lib/ProfileData/InstrProfWriter.cpp b/llvm/lib/ProfileData/InstrProfWriter.cpp new file mode 100644 index 00000000000..320860a4cb4 --- /dev/null +++ b/llvm/lib/ProfileData/InstrProfWriter.cpp @@ -0,0 +1,58 @@ +//=-- InstrProfWriter.cpp - Instrumented profiling writer -------------------=// +// +// The LLVM Compiler Infrastructure +// +// This file is distributed under the University of Illinois Open Source +// License. See LICENSE.TXT for details. +// +//===----------------------------------------------------------------------===// +// +// This file contains support for writing profiling data for clang's +// instrumentation based PGO and coverage. +// +//===----------------------------------------------------------------------===// + +#include "llvm/ProfileData/InstrProfWriter.h" +#include "llvm/Support/Endian.h" + +using namespace llvm; + +error_code InstrProfWriter::addFunctionCounts(StringRef FunctionName, + uint64_t FunctionHash, + ArrayRef<uint64_t> Counters) { + auto Where = FunctionData.find(FunctionName); + if (Where == FunctionData.end()) { + // If this is the first time we've seen this function, just add it. + FunctionData[FunctionName] = {FunctionHash, Counters}; + return instrprof_error::success;; + } + + auto &Data = Where->getValue(); + // We can only add to existing functions if they match, so we check the hash + // and number of counters. + if (Data.Hash != FunctionHash) + return instrprof_error::hash_mismatch; + if (Data.Counts.size() != Counters.size()) + return instrprof_error::count_mismatch; + // These match, add up the counters. + for (size_t I = 0, E = Counters.size(); I < E; ++I) { + if (Data.Counts[I] + Counters[I] < Data.Counts[I]) + return instrprof_error::counter_overflow; + Data.Counts[I] += Counters[I]; + } + return instrprof_error::success; +} + +void InstrProfWriter::write(raw_ostream &OS) { + // Write out the counts for each function. + for (const auto &I : FunctionData) { + StringRef Name = I.getKey(); + uint64_t Hash = I.getValue().Hash; + const std::vector<uint64_t> &Counts = I.getValue().Counts; + + OS << Name << "\n" << Hash << "\n" << Counts.size() << "\n"; + for (uint64_t Count : Counts) + OS << Count << "\n"; + OS << "\n"; + } +} |

