[llvm] [llvm-dwarfdump][LineCov 1/3] Add variable coverage metrics (PR #169646)
Stephen Tozer via llvm-commits
llvm-commits at lists.llvm.org
Mon Dec 1 06:29:31 PST 2025
https://github.com/SLTozer commented:
Some comments from my look over of this patch - due to the size of the patch (understandable since it's adding and testing the smallest complete form of this feature) it might take a couple of passes to complete review.
Also since this is an implementation of the RFC that you've linked to in the PR description, it'd be a good idea to comment on the RFC thread linking to this PR so that those who participated in or watched that thread can follow through to the implementation.
Also on that note it'd be good to hear from @adrian-prantl - do you have any thoughts on this feature and its implementation, and/or have any suggested reviewers who may be more familiar with dwarfdump than me?
https://github.com/llvm/llvm-project/pull/169646
More information about the llvm-commits
mailing list