[PATCH] D11933: Extend debug ranges and provide multiple location support for debug variables
Vikram TV via llvm-commits
llvm-commits at lists.llvm.org
Wed Aug 12 01:02:46 PDT 2015
tvvikram added a comment.
In http://reviews.llvm.org/D11933#221791, @dblaikie wrote:
> +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)?
I am separating the patches.
http://reviews.llvm.org/D11933
More information about the llvm-commits
mailing list