[PATCH] D23277: [llvm-cov] Add the "Goto first zero count" feature.

Vedant Kumar via llvm-commits llvm-commits at lists.llvm.org
Fri Aug 26 11:25:07 PDT 2016


vsk added a comment.

Thanks for splitting things up :).

I'm not sure that the javascript bit is needed. It looks like a way to remove/disable a link, but I don't see any issue with leaving the link active.

I also don't think we need a 'zeroline' or 'zero' id. Every line already has an anchor: <a name='L$N'>. I think a "Jump to the first uncovered line" link which uses one of the line anchors would do the job.


https://reviews.llvm.org/D23277





More information about the llvm-commits mailing list