Page MenuHomePhabricator

[sancov] Ignore PC samples with value 0

Authored by mcgrathr on Sun, Jun 23, 3:58 PM.



The sancov runtime for the (Fuchsia) Zircon kernel delivers results
in the standard format, but as the full array of possible samples
with 0 in uncovered slots. That runtime delivers "live" data and
has no final "export" pass to compactify out the uncovered slots,
and it seems silly to require another offline tool just for that.

Diff Detail


Event Timeline

mcgrathr created this revision.Sun, Jun 23, 3:58 PM
Herald added a project: Restricted Project. · View Herald TranscriptSun, Jun 23, 3:58 PM has the runtime implementation that motivated this.

MaskRay added inline comments.Mon, Jun 24, 2:48 AM
265 ↗(On Diff #206142)

The erase(0) logic should probably be moved here.

mcgrathr updated this revision to Diff 206277.Mon, Jun 24, 12:22 PM

Move logic out of RawCoverage constructor.

mcgrathr marked an inline comment as done.Mon, Jun 24, 12:23 PM
phosek accepted this revision.Sat, Jun 29, 1:12 PM


This revision is now accepted and ready to land.Sat, Jun 29, 1:12 PM
MaskRay accepted this revision.Mon, Jul 1, 8:09 PM
This revision was automatically updated to reflect the committed changes.