diff options
author | Vedant Kumar <vsk@apple.com> | 2016-03-28 20:12:07 +0000 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2016-03-28 20:12:07 +0000 |
commit | f20b6cec1c50d624fb5fb6a9003407be6e9f1bad (patch) | |
tree | 72f1570255b05eba3bbcaac3faaf05b41c4b89f4 /llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l | |
parent | 0d026d9e9eb4fa1695dc94be43f02579f59bc696 (diff) | |
download | bcm5719-llvm-f20b6cec1c50d624fb5fb6a9003407be6e9f1bad.tar.gz bcm5719-llvm-f20b6cec1c50d624fb5fb6a9003407be6e9f1bad.zip |
Reapply "[PGO] Fix name encoding for ObjC-like functions"
Function names in ObjC can have spaces in them. This interacts poorly
with name compression, which uses spaces to separate PGO names. Fix the
issue by using a different separator and update a test.
I chose "\01" as the separator because 1) it's non-printable, 2) we
strip it from PGO names, and 3) it's the next natural choice once "\00"
is discarded (that one's overloaded).
This reverts the revert commit beaf3d18. What's changed?
- I fixed up the covmap-V2 binary format tests using a linux VM.
- I updated the expected counts in instrprof-comdat.h to account for
the fact that there have been bugfixes to clang coverage.
- I added an assert to make sure we don't get bitten by this again.
Differential Revision: http://reviews.llvm.org/D18516
llvm-svn: 264641
Diffstat (limited to 'llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l')
-rwxr-xr-x | llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l | bin | 2125077 -> 2124848 bytes |
1 files changed, 0 insertions, 0 deletions
diff --git a/llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l b/llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l Binary files differindex af4d39cedfb..9c6835cdff1 100755 --- a/llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l +++ b/llvm/test/tools/llvm-cov/Inputs/binary-formats.v2.linux64l |