[PATCH] D36020: [llvm-cov] Unify region marker placement between text/html modes

Vedant Kumar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jul 28 14:24:56 PDT 2017


vsk created this revision.

Make sure that the text and html emitters always emit the same set of
region markers, and avoid emitting redundant markers for line segments
which don't end on the line they start on.

Here is the html output from the deferred-region.cpp test:
F3813435: region-markers.html <https://reviews.llvm.org/F3813435>

This is related to https://reviews.llvm.org/D35925, and depends on https://reviews.llvm.org/D36014


https://reviews.llvm.org/D36020

Files:
  test/tools/llvm-cov/deferred-region.cpp
  test/tools/llvm-cov/showRegionMarkers.cpp
  tools/llvm-cov/CodeCoverage.cpp
  tools/llvm-cov/CoverageViewOptions.h
  tools/llvm-cov/SourceCoverageView.cpp
  tools/llvm-cov/SourceCoverageView.h
  tools/llvm-cov/SourceCoverageViewHTML.cpp
  tools/llvm-cov/SourceCoverageViewText.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D36020.108714.patch
Type: text/x-patch
Size: 9759 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170728/d461b1ac/attachment.bin>


More information about the llvm-commits mailing list