[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