This is an archive of the discontinued LLVM Phabricator instance.

[llvm-cov] Skip unnecessary coverage computations for "export -summary-only".
ClosedPublic

Authored by Dor1s on Jan 12 2018, 10:28 AM.

Details

Summary

This speeds up export "summary-only" execution by an order of magnitude or two,
depending on number of threads used for prepareFileReports execution.

Also includes minor refactoring for splitting render of summary and detailed data
in two independent methods.

Event Timeline

Dor1s created this revision.Jan 12 2018, 10:28 AM
Dor1s edited the summary of this revision. (Show Details)Jan 12 2018, 10:29 AM
Dor1s updated this revision to Diff 129664.Jan 12 2018, 10:47 AM

Refactor methods for file rendering

Dor1s updated this revision to Diff 129665.Jan 12 2018, 10:51 AM

Clean up the code

Dor1s edited the summary of this revision. (Show Details)Jan 12 2018, 10:53 AM
Dor1s added reviewers: vsk, morehouse.
Dor1s added subscribers: kcc, llvm-commits.
vsk accepted this revision.Jan 12 2018, 12:21 PM

Looks good, thanks!

This revision is now accepted and ready to land.Jan 12 2018, 12:21 PM
Dor1s closed this revision.Jan 12 2018, 12:32 PM