[PATCH] D20286: [Coverage] Fix an issue where improper coverage mapping data could be loaded for an inline function.

Igor Kudrin via llvm-commits llvm-commits at lists.llvm.org
Wed May 18 08:23:42 PDT 2016


ikudrin updated the summary for this revision.
ikudrin updated this revision to Diff 57632.
ikudrin added a comment.

- Added the class to check if the coverage mapping data is dummy or real.
- New implementations also works for simple functions with zero hash, so http://reviews.llvm.org/D20287 is not needed anymore.
- The test case was extended to check simple inline functions in addition to regular ones.


http://reviews.llvm.org/D20286

Files:
  include/llvm/ProfileData/Coverage/CoverageMappingReader.h
  lib/ProfileData/Coverage/CoverageMappingReader.cpp
  test/tools/llvm-cov/Inputs/prefer_used_to_unused.covmapping
  test/tools/llvm-cov/Inputs/prefer_used_to_unused.cpp
  test/tools/llvm-cov/Inputs/prefer_used_to_unused.proftext
  test/tools/llvm-cov/prefer_used_to_unused.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D20286.57632.patch
Type: text/x-patch
Size: 8528 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160518/40ca17fa/attachment.bin>


More information about the llvm-commits mailing list