[PATCH] D23277: Improves the HTML report for the source file.

Ying Yi via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 8 14:08:52 PDT 2016


MaggieYi added a comment.

In https://reviews.llvm.org/D23277#508994, @vsk wrote:

> On a high-level, the changes you've made look good. However, I think that this
>  patch needs to be split up into 3 smaller patches for each functional change
>  you listed in your summary. That would make it easier for me to review it, and
>  would make the commit history easier to read through.


Many thanks for your comments. I am happy to split up this patch into 3 smaller patches. 
I have put the first patch into Phabricator (https://reviews.llvm.org/D23281).

I will update the rest of patch following your comments.


https://reviews.llvm.org/D23277





More information about the llvm-commits mailing list