InstrProf: Allow multiple functions with the same name
authorJustin Bogner <mail@justinbogner.com>
Fri, 1 Aug 2014 22:50:07 +0000 (22:50 +0000)
committerJustin Bogner <mail@justinbogner.com>
Fri, 1 Aug 2014 22:50:07 +0000 (22:50 +0000)
commit821d7471f91858ce91de86df940cedfab4166543
tree2fcdad0febecb3f9cac05d44dad329d0759b64a9
parent6d3adac2179bc7867f55d108111ec38877b8b7c0
InstrProf: Allow multiple functions with the same name

This updates the instrumentation based profiling format so that when
we have multiple functions with the same name (but different function
hashes) we keep all of them instead of rejecting the later ones.

There are a number of scenarios where this can come up where it's more
useful to keep multiple function profiles:

* Name collisions in unrelated libraries that are profiled together.
* Multiple "main" functions from multiple tools built against a common
  library.
* Combining profiles from different build configurations (ie, asserts
  and no-asserts)

The profile format now stores the number of counters between the hash
and the counts themselves, so that multiple sets of counts can be
stored. Since this is backwards incompatible, I've bumped the format
version and added some trivial logic to skip this when reading the old
format.

llvm-svn: 214585
llvm/include/llvm/ProfileData/InstrProfReader.h
llvm/include/llvm/ProfileData/InstrProfWriter.h
llvm/lib/ProfileData/InstrProfIndexed.h
llvm/lib/ProfileData/InstrProfReader.cpp
llvm/lib/ProfileData/InstrProfWriter.cpp
llvm/test/tools/llvm-profdata/Inputs/compat.profdata.v1 [new file with mode: 0644]
llvm/test/tools/llvm-profdata/compat.proftext [new file with mode: 0644]
llvm/test/tools/llvm-profdata/hash-mismatch.proftext