[PATCH] D36298: [llvm-cov] Rearrange entries in HTML report index.

[Vedant Kumar via Phabricator via llvm-commits]

vsk added a comment.

This seems like a clean solution. The total file coverage summaries aren't affected, which is nice.

Could you make a similar change to CoverageReport::prepareFileReports? We try to maintain parity between the text/html emitters, and the text emitter uses CoverageReport to prepare its index file. This should do it: F4295704: CoverageReport.patch <https://reviews.llvm.org/F4295704>.

You could add a RUN line to the test you've updated here, and use -format text, to get a test case. None of the other tests appear to be affected.


Repository:
  rL LLVM

https://reviews.llvm.org/D36298