[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