[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