diff options
author | Vedant Kumar <vsk@apple.com> | 2016-06-29 00:38:21 +0000 |
---|---|---|
committer | Vedant Kumar <vsk@apple.com> | 2016-06-29 00:38:21 +0000 |
commit | 8d74cb27e82440dbf1bea123613fc373e203680a (patch) | |
tree | 2e22c0a2ccf2992092030a25440288f71a1a2513 /llvm/tools/llvm-cov/SourceCoverageViewText.cpp | |
parent | 1ead14b147c5aecacc25d6f1432c0a76b4b91e8c (diff) | |
download | llvm-8d74cb27e82440dbf1bea123613fc373e203680a.zip llvm-8d74cb27e82440dbf1bea123613fc373e203680a.tar.gz llvm-8d74cb27e82440dbf1bea123613fc373e203680a.tar.bz2 |
[llvm-cov] Minor cleanups to prepare for the html format patch
- Add renderView{Header,Footer}, renderLineSuffix, and hasSubViews to
support creating tables with nested views.
- Move the 'Format' cl::opt to make it easier to extend.
- Just create one function view file, instead of overwriting the same
file for every new function. Add a regression test for this.
llvm-svn: 274086
Diffstat (limited to 'llvm/tools/llvm-cov/SourceCoverageViewText.cpp')
-rw-r--r-- | llvm/tools/llvm-cov/SourceCoverageViewText.cpp | 13 |
1 files changed, 11 insertions, 2 deletions
diff --git a/llvm/tools/llvm-cov/SourceCoverageViewText.cpp b/llvm/tools/llvm-cov/SourceCoverageViewText.cpp index 848b5f8..70ded87 100644 --- a/llvm/tools/llvm-cov/SourceCoverageViewText.cpp +++ b/llvm/tools/llvm-cov/SourceCoverageViewText.cpp @@ -59,6 +59,10 @@ unsigned getDividerWidth(const CoverageViewOptions &Opts) { } // anonymous namespace +void SourceCoverageViewText::renderViewHeader(raw_ostream &OS) { (void)OS; } + +void SourceCoverageViewText::renderViewFooter(raw_ostream &OS) { (void)OS; } + void SourceCoverageViewText::renderSourceName(raw_ostream &OS) { getOptions().colored_ostream(OS, raw_ostream::CYAN) << getSourceName() << ":\n"; @@ -70,6 +74,12 @@ void SourceCoverageViewText::renderLinePrefix(raw_ostream &OS, OS << " |"; } +void SourceCoverageViewText::renderLineSuffix(raw_ostream &OS, + unsigned ViewDepth) { + (void)OS; + (void)ViewDepth; +} + void SourceCoverageViewText::renderViewDivider(raw_ostream &OS, unsigned ViewDepth) { assert(ViewDepth != 0 && "Cannot render divider at top level"); @@ -180,8 +190,7 @@ void SourceCoverageViewText::renderRegionMarkers( } void SourceCoverageViewText::renderExpansionSite( - raw_ostream &OS, ExpansionView &ESV, LineRef L, - const coverage::CoverageSegment *WrappedSegment, + raw_ostream &OS, LineRef L, const coverage::CoverageSegment *WrappedSegment, CoverageSegmentArray Segments, unsigned ExpansionCol, unsigned ViewDepth) { renderLinePrefix(OS, ViewDepth); OS.indent(getCombinedColumnWidth(getOptions()) + (ViewDepth == 0 ? 0 : 1)); |