[PATCH] D151283: [llvm-cov] Support a Hierarchical Structure for HTML Coverage Report Generating

[Gulfem Savrun Yeniceri via Phabricator via llvm-commits]

gulfem added a comment.

Can we simplify the commit message? We do not need all the details in the commit message as we have them in the Discourse discussion. 
I would suggest something like: "llvm-cov currently generates a single top-level index HTML file, which causes rendering scalability issues in large projects. This patch adds support for hierarchical directory structure into the HTML reports to solve scalability issues."



================
Comment at: llvm/docs/CommandGuide/llvm-cov.rst:256
+ Defaults to false and a single index file will be generated in the output
+ directory (which can be very large and impossible to read for large projects).
+
----------------
How about we stop the description before the parenthesis? I think "impossible to read" is a strong statement.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D151283