[PATCH] D23281: [LLVMCov] Swapped the line and count columns.

Ying Yi via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 9 11:37:43 PDT 2016


MaggieYi added a comment.

In https://reviews.llvm.org/D23281#510048, @vsk wrote:

> I think it would be fine to commit the two patches separately but simultaneously. The bots may go red temporarily, but that won't be alarming if you mention that a follow-up commit in compiler-rt is needed in your commit message.
>
> Do you have a linux machine to test the changes to test/profile/Linux/coverage_*? If not, I can help with that.


Thanks very much for your help. I have a linux machine to run the compiler-rt tests. With the change, all llvm/clang/compiler-rt regression tests passed.

I will follow your advice to commit two patches separately but simultaneously.


https://reviews.llvm.org/D23281





More information about the llvm-commits mailing list