This is an archive of the discontinued LLVM Phabricator instance.

[llvm-dwarfdump][Statistics] Change the coverage buckets representation. NFC
ClosedPublic

Authored by krisb on Dec 11 2019, 10:14 AM.

Details

Summary

This changes the representation of 'coverage buckets' in llvm-dwarfdump and
llvm-locstats to one that makes more clear what the buckets contain.

See some related details in D71070.

Event Timeline

krisb created this revision.Dec 11 2019, 10:14 AM
Herald added a project: Restricted Project. · View Herald Transcript
krisb updated this revision to Diff 233406.Dec 11 2019, 10:31 AM

Fix indentation in llvm-locstats output.

@krisb Thanks for this!

This one will go before the D71070? Can we make a stack of the patches?

llvm/docs/CommandGuide/llvm-locstats.rst
58

[1%,10%) ?

djtodoro added a subscriber: asowda.
krisb updated this revision to Diff 233517.Dec 11 2019, 11:42 PM
krisb marked an inline comment as done.

Fixed a misprint in llvm/docs/CommandGuide/llvm-locstats.rst.

@djtodoro , Ah, missed this one. Thank you so much!

krisb updated this revision to Diff 233528.Dec 12 2019, 1:10 AM

Fixed one more misprint.

djtodoro accepted this revision.Dec 12 2019, 4:54 AM

LGTM! Thanks.

Please wait for someone else from the debug-info project to take a look as well.

This revision is now accepted and ready to land.Dec 12 2019, 4:54 AM
aprantl accepted this revision.Dec 12 2019, 9:16 AM
This revision was automatically updated to reflect the committed changes.