[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