en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata#86705ba5 Reapply (2x) "[PGO] Fix name encoding for ObjC-like functions"Function names in ObjC can have spaces in them. This interacts poorlywith name compression, which uses spaces to separate PGO names. Fix theissue by using a different separator and update a test.I chose "\01" as the separator because 1) it's non-printable, 2) westrip it from PGO names, and 3) it's the next natural choice once "\00"is discarded (that one's overloaded).What's changed since the original commit?- I fixed up the covmap-V2 binary format tests using a linux VM.- I weakened the CHECK lines in instrprof-comdat.h to account for the fact that there have been bugfixes to clang coverage. These will be fixed up in a follow-up.- I added an assert to make sure we don't get bitten by this again.- I constructed the c-general.profraw file without name compression enabled to appease some bots.Differential Revision: http://reviews.llvm.org/D18516llvm-svn: 264658 List of files: /llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata Mon, 28 Mar 2016 21:06:42 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata#476a94d9 Revert "Reapply "[PGO] Fix name encoding for ObjC-like functions""This reverts commit r264641 to investigate why c-general.test is failingon the bots.llvm-svn: 264643 List of files: /llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata Mon, 28 Mar 2016 20:20:40 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata#f20b6cec Reapply "[PGO] Fix name encoding for ObjC-like functions"Function names in ObjC can have spaces in them. This interacts poorlywith name compression, which uses spaces to separate PGO names. Fix theissue by using a different separator and update a test.I chose "\01" as the separator because 1) it's non-printable, 2) westrip 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/D18516llvm-svn: 264641 List of files: /llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata Mon, 28 Mar 2016 20:12:07 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata#b4fc4cbe re-submit test case (withright format-version)llvm-svn: 258384 List of files: /llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata Thu, 21 Jan 2016 02:35:59 +0000 Xinliang David Li <davidxl@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata#bc05e4e4 [Coverage] Add a test case for comdatThe binary contains two (merged) covmap sections whichhave duplicate CovMapRecords from comdat (template instantation).This test makes sure the reader reads it properly. It alsotests that the coverage data from different instantiationsof the same template function are properly merged in showoutput.llvm-svn: 258376 List of files: /llvm-project-15.0.7/llvm/test/tools/llvm-cov/Inputs/elf_binary_comdat.profdata Thu, 21 Jan 2016 00:57:42 +0000 Xinliang David Li <davidxl@google.com>