en Copyright 2015 Java http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#dd1ea9de Reland: [Coverage] Revise format to reduce binary sizeTry again with an up-to-date version of D69471 (99317124 was a stalerevision).---Revise the coverage mapping format to reduce binary size by:1. Naming function records and marking them `linkonce_odr`, and2. Compressing filenames.This shrinks the size of llc's coverage segment by 82% (334MB -> 62MB)and speeds up end-to-end single-threaded report generation by 10%. Forreference the compressed name data in llc is 81MB (__llvm_prf_names).Rationale for changes to the format:- With the current format, most coverage function records are discarded. E.g., more than 97% of the records in llc are *duplicate* placeholders for functions visible-but-not-used in TUs. Placeholders *are* used to show under-covered functions, but duplicate placeholders waste space.- We reached general consensus about giving (1) a try at the 2017 code coverage BoF [1]. The thinking was that using `linkonce_odr` to merge duplicates is simpler than alternatives like teaching build systems about a coverage-aware database/module/etc on the side.- Revising the format is expensive due to the backwards compatibility requirement, so we might as well compress filenames while we're at it. This shrinks the encoded filenames in llc by 86% (12MB -> 1.6MB).See CoverageMappingFormat.rst for the details on what exactly haschanged.Fixes PR34533 [2], hopefully.[1] http://lists.llvm.org/pipermail/llvm-dev/2017-October/118428.html[2] https://bugs.llvm.org/show_bug.cgi?id=34533Differential Revision: https://reviews.llvm.org/D69471 List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Mon, 21 Oct 2019 18:48:38 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#33888717 Revert "[Coverage] Revise format to reduce binary size"This reverts commit 99317124e1c772e9a9de41a0cd56e1db049b4ea4. This isstill busted on Windows:http://lab.llvm.org:8011/builders/lld-x86_64-win7/builds/40873The llvm-cov tests report 'error: Could not load coverage information'. List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Sat, 29 Feb 2020 02:03:15 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#99317124 [Coverage] Revise format to reduce binary sizeRevise the coverage mapping format to reduce binary size by:1. Naming function records and marking them `linkonce_odr`, and2. Compressing filenames.This shrinks the size of llc's coverage segment by 82% (334MB -> 62MB)and speeds up end-to-end single-threaded report generation by 10%. Forreference the compressed name data in llc is 81MB (__llvm_prf_names).Rationale for changes to the format:- With the current format, most coverage function records are discarded. E.g., more than 97% of the records in llc are *duplicate* placeholders for functions visible-but-not-used in TUs. Placeholders *are* used to show under-covered functions, but duplicate placeholders waste space.- We reached general consensus about giving (1) a try at the 2017 code coverage BoF [1]. The thinking was that using `linkonce_odr` to merge duplicates is simpler than alternatives like teaching build systems about a coverage-aware database/module/etc on the side.- Revising the format is expensive due to the backwards compatibility requirement, so we might as well compress filenames while we're at it. This shrinks the encoded filenames in llc by 86% (12MB -> 1.6MB).See CoverageMappingFormat.rst for the details on what exactly haschanged.Fixes PR34533 [2], hopefully.[1] http://lists.llvm.org/pipermail/llvm-dev/2017-October/118428.html[2] https://bugs.llvm.org/show_bug.cgi?id=34533Differential Revision: https://reviews.llvm.org/D69471 List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Mon, 21 Oct 2019 18:48:38 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#f208b70f Revert "[Coverage] Revise format to reduce binary size"This reverts commit e18531595bba495946aa52c0a16b9f9238cff8bc.On Windows, there is an error:http://lab.llvm.org:8011/builders/sanitizer-windows/builds/54963/steps/stage%201%20check/logs/stdioerror: C:\b\slave\sanitizer-windows\build\stage1\projects\compiler-rt\test\profile\Profile-x86_64\Output\instrprof-merging.cpp.tmp.v1.o: Failed to load coverage: Malformed coverage data List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Wed, 04 Dec 2019 18:35:14 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#e1853159 [Coverage] Revise format to reduce binary sizeRevise the coverage mapping format to reduce binary size by:1. Naming function records and marking them `linkonce_odr`, and2. Compressing filenames.This shrinks the size of llc's coverage segment by 82% (334MB -> 62MB)and speeds up end-to-end single-threaded report generation by 10%. Forreference the compressed name data in llc is 81MB (__llvm_prf_names).Rationale for changes to the format:- With the current format, most coverage function records are discarded. E.g., more than 97% of the records in llc are *duplicate* placeholders for functions visible-but-not-used in TUs. Placeholders *are* used to show under-covered functions, but duplicate placeholders waste space.- We reached general consensus about giving (1) a try at the 2017 code coverage BoF [1]. The thinking was that using `linkonce_odr` to merge duplicates is simpler than alternatives like teaching build systems about a coverage-aware database/module/etc on the side.- Revising the format is expensive due to the backwards compatibility requirement, so we might as well compress filenames while we're at it. This shrinks the encoded filenames in llc by 86% (12MB -> 1.6MB).See CoverageMappingFormat.rst for the details on what exactly haschanged.Fixes PR34533 [2], hopefully.[1] http://lists.llvm.org/pipermail/llvm-dev/2017-October/118428.html[2] https://bugs.llvm.org/show_bug.cgi?id=34533Differential Revision: https://reviews.llvm.org/D69471 List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Mon, 21 Oct 2019 18:48:38 +0000 Vedant Kumar <vsk@apple.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#13cf09dc Test simplificationllvm-svn: 261047 List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Wed, 17 Feb 2016 00:59:01 +0000 Xinliang David Li <davidxl@google.com> http://172.16.0.5:8080/history/llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp#a951e8ec [PGO] Fix issue: explicitly defaulted assignop is not profiledDifferential Revision: http://reviews.llvm.org/D16947 llvm-svn: 260270 List of files: /llvm-project-15.0.7/clang/test/Profile/def-assignop.cpp Tue, 09 Feb 2016 20:02:59 +0000 Xinliang David Li <davidxl@google.com>