[llvm] [llvm-cov][CoverageView] minor fix/improvement to HTML and text coverage output (PR #80952)

Alan Phipps via llvm-commits llvm-commits at lists.llvm.org
Tue Feb 13 11:33:02 PST 2024


evodius96 wrote:

> A gentle bump for more comments. (Which members are the best to include?)

Your fix is minor and straightforward; I don't see the need for additional reviewers, but it's not a problem.

> By the way, @evodius96 do you have plans of updating this page https://llvm.org/docs/CoverageMappingFormat.html ? We probably want to include the changes since version 6. I can also do a draft for that.

You're right -- the document needs to be updated for version 7.  I did add information about the Decision mapping region in the Mapping Region section, but the discussion of what changed between version 6 and 7 also needs to mention that.  If you want to draft that change, that's OK with me.

> Also could you please take a look at https://github.com/llvm/llvm-project/pull/80531?

I can take a look, but it does look like you have the right reviewers on it.

https://github.com/llvm/llvm-project/pull/80952


More information about the llvm-commits mailing list