<div dir="ltr">+samsonov, who built DbgValueHistoryCalculator<br><br>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)<br><br>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)?</div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Aug 11, 2015 at 3:34 AM, Keno Fischer <span dir="ltr"><<a href="mailto:kfischer@college.harvard.edu" target="_blank">kfischer@college.harvard.edu</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">loladiro added reviewers: aprantl, dblaikie, friss, echristo.<br>
loladiro added a comment.<br>
<br>
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.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D11933" rel="noreferrer" target="_blank">http://reviews.llvm.org/D11933</a><br>
<br>
<br>
<br>
</blockquote></div><br></div>