[PATCH] D71070: [llvm-dwarfdump][Statistics] Don't count coverage less than 1% as 0%

Kristina Bessonova via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 9 03:15:51 PST 2019


krisb added a comment.

@dblaikie , @djtodoro sorry for paying so much attention to details here, but does this one work with you?

       cov%          samples       percentage(~)
  -------------------------------------------------
     0%                    1              16%
     ( 0, 10%)             0               0%
     [10, 20%)             0               0%
     [20, 30%)             0               0%
     [30, 40%)             0               0%
     [40, 50%)             0               0%
     [50, 60%)             1              16%
     [60, 70%)             0               0%
     [70, 80%)             0               0%
     [80, 90%)             1              16%
     [90,100%)             0               0%
     100%                  3              50%

And just mention here how the previously suggested option looks like:

        cov%          samples       percentage(~)
  -------------------------------------------------
     No location info      1              16%
     0-9%                  0               0%
     10-19%                0               0%
     20-29%                0               0%
     30-39%                0               0%
     40-49%                0               0%
     50-59%                1              16%
     60-69%                0               0%
     70-79%                0               0%
     80-89%                1              16%
     90-99%                0               0%
     100%                  3              50%


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D71070/new/

https://reviews.llvm.org/D71070





More information about the llvm-commits mailing list