[PATCH] D24687: [llvm-cov] Add a project coverage summary above the index page.

Ying Yi via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 16 15:12:33 PDT 2016


MaggieYi created this revision.
MaggieYi added reviewers: vsk, davidxl, bogner.
MaggieYi added a subscriber: llvm-commits.

Currently, the project coverage summary is given in the bottom row of the index table. If there are a large number of source files listed in the index table, the summary row is not immediately visible. This patch adds the project coverage summary above the index page to improve the report readability. I have also added the project directory to the summary.

In addition, filenames are only shown as a part of path name in the filename column of the index page.  If the file path includes many directory levels, this column doesnÎéÎ÷t look nice. This patch separates the filename and path into two columns, and the path is shown the relative to the project directory specified in the summary, which also improves clarity.

https://reviews.llvm.org/D24687

Files:
  test/tools/llvm-cov/multiple-files.test
  test/tools/llvm-cov/native_separators.c
  test/tools/llvm-cov/showLineExecutionCounts.cpp
  tools/llvm-cov/SourceCoverageViewHTML.cpp
  tools/llvm-cov/SourceCoverageViewHTML.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D24687.71705.patch
Type: text/x-patch
Size: 11634 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160916/8a4f5fa2/attachment.bin>


More information about the llvm-commits mailing list