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/lib/Transforms/Utils/LoopUnrollRuntime.cpp | |
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/lib/Transforms/Utils/LoopUnrollRuntime.cpp')
0 files changed, 0 insertions, 0 deletions