[PATCH] D11933: Extend debug ranges and provide multiple location support for debug variables

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 11 09:52:07 PDT 2015


+samsonov, who built DbgValueHistoryCalculator

It might be helpful to have a bit of a high level design describing how the
patch achieves this (you could go through & write comments in the code
review to describe the various parts - if they're not worthy of
sufficiently descriptive inline comments in the source itself)

Is it reasonable/possible/helpful to split the two (A and B, in your
original description) features into separate patches to simplify review
(smaller patches are exponentially easier/faster to review)?

On Tue, Aug 11, 2015 at 3:34 AM, Keno Fischer <kfischer at college.harvard.edu>
wrote:

> loladiro added reviewers: aprantl, dblaikie, friss, echristo.
> loladiro added a comment.
>
> Adding a few more people who I'd like to take a look and comment. For
> background, this work is being done as part of our effort to improve debug
> info quality in julia. There is still some work left to do, but I'd like to
> gather feedback on implementation/design early, as well as make people
> aware that this is being worked on.
>
>
> http://reviews.llvm.org/D11933
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150811/211f5121/attachment.html>


More information about the llvm-commits mailing list