diff options
author | Vedant Kumar <vsk@apple.com> | 2017-09-18 23:37:28 +0000 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2017-09-18 23:37:28 +0000 |
commit | ad8f637bd83aeeca7321d6c74b3d7787587c0d55 (patch) | |
tree | e771084274c8fde30518a8c9f08c7ffb4e2cd3dd /llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp | |
parent | 27d1b14ab056a3de64cf93c80a10886e2dc904f7 (diff) | |
download | llvm-ad8f637bd83aeeca7321d6c74b3d7787587c0d55.zip llvm-ad8f637bd83aeeca7321d6c74b3d7787587c0d55.tar.gz llvm-ad8f637bd83aeeca7321d6c74b3d7787587c0d55.tar.bz2 |
[Coverage] Use gap regions to select better line exec counts
After clang started emitting deferred regions (r312818), llvm-cov has
had a hard time picking reasonable line execuction counts. There have
been one or two generic improvements in this area (e.g r310012), but
line counts can still report coverage for whitespace instead of code
(llvm.org/PR34612).
To fix the problem:
* Introduce a new region kind so that frontends can explicitly label
gap areas.
This is done by changing the encoding of the columnEnd field of
MappingRegion. This doesn't substantially increase binary size, and
makes it easy to maintain backwards-compatibility.
* Don't set the line count to a count from a gap area, unless the count
comes from a wrapped segment.
* Don't highlight gap areas as uncovered.
Fixes llvm.org/PR34612.
llvm-svn: 313597
Diffstat (limited to 'llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp')
-rw-r--r-- | llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp b/llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp index 1a21b72..300a016 100644 --- a/llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp +++ b/llvm/tools/llvm-cov/SourceCoverageViewHTML.cpp @@ -527,7 +527,7 @@ void SourceCoverageViewHTML::renderLine( const auto *CurSeg = Segments[I]; if (CurSeg->Col == ExpansionCol) Color = "cyan"; - else if (CheckIfUncovered(CurSeg)) + else if (!CurSeg->IsGapRegion && CheckIfUncovered(CurSeg)) Color = "red"; else Color = None; |