[PATCH] D18278: llvm-cov HTML Generation

[Ying Yi via llvm-commits]

MaggieYi added a comment.

In http://reviews.llvm.org/D18278#474741, @vsk wrote:

> Thanks for testing it out!
>
> It seems that all the pending issues we are discussing actually relate to `CoveragePrinter::getOutputPath`, which is not a part of this patch. We can fix them before or after this patch is committed, and test the changes using "-format text -output-dir ...".
>
> Given that this is a large change, I'd like to make sure it addresses Justin's concerns and that it has an explicit lgtm before moving forward.


I agree with you, we could fix these issues after the patch is committed.

The patch in the current state looks good to me.


http://reviews.llvm.org/D18278