Page MenuHomePhabricator

[llvm-cov] Add the project summary to the text coverage report for each source file.
ClosedPublic

Authored by MaggieYi on Sep 5 2016, 1:21 PM.

Diff Detail

Repository
rL LLVM

Event Timeline

MaggieYi updated this revision to Diff 70346.Sep 5 2016, 1:21 PM
MaggieYi retitled this revision from to [llvm-cov] Add the project summary to the text coverage report for each source file..
MaggieYi updated this object.
MaggieYi added reviewers: vsk, davidxl, bogner.
MaggieYi added a subscriber: llvm-commits.
vsk accepted this revision.Sep 6 2016, 6:55 AM
vsk edited edge metadata.

Thanks for doing this. LGTM with one small change.

tools/llvm-cov/SourceCoverageView.h
277 ↗(On Diff #70346)

Please add a short doxygen comment here, e.g 'Return the source name formatted for the host OS.'

This revision is now accepted and ready to land.Sep 6 2016, 6:55 AM
This revision was automatically updated to reflect the committed changes.