aboutsummaryrefslogtreecommitdiff
path: root/clang/lib/Frontend/CompilerInvocation.cpp
diff options
context:
space:
mode:
authorserge-sans-paille <sguelton@redhat.com>2020-05-25 20:44:35 +0200
committerserge-sans-paille <sguelton@redhat.com>2020-05-27 09:15:21 +0200
commitde02a75e398415bad4df27b4547c25b896c8bf3b (patch)
tree7b8d4f4ff94a8dd23414fe3a399fc40518755a5e /clang/lib/Frontend/CompilerInvocation.cpp
parent84cf8ed8fd3f950b6e30225cae6f092da768cbe6 (diff)
downloadllvm-de02a75e398415bad4df27b4547c25b896c8bf3b.zip
llvm-de02a75e398415bad4df27b4547c25b896c8bf3b.tar.gz
llvm-de02a75e398415bad4df27b4547c25b896c8bf3b.tar.bz2
[PGO] Fix computation of function Hash
And bump its version number accordingly. This is a patched recommit of 7c298c104bfe725d4315926a656263e8a5ac3054 Previous hash implementation was incorrectly passing an uint64_t, that got converted to an uint8_t, to finalize the hash computation. This led to different functions having the same hash if they only differ by the remaining statements, which is incorrect. Added a new test case that trivially tests that a small function change is reflected in the hash value. Not that as this patch fixes the hash computation, it would invalidate all hashes computed before that patch applies, this is why we bumped the version number. Update profile data hash entries due to hash function update, except for binary version, in which case we keep the buggy behavior for backward compatibility. Differential Revision: https://reviews.llvm.org/D79961
Diffstat (limited to 'clang/lib/Frontend/CompilerInvocation.cpp')
0 files changed, 0 insertions, 0 deletions