Benjamin,<div><br></div><div>Could you take a look at this? (or redirect me to someone else for review).</div><div><br><div class="gmail_quote">On Wed, Jul 25, 2012 at 9:15 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div>Could you please take a look?</div><div><br></div><div>(re-attaching the patch just in case)</div><div class="HOEnZb">
<div class="h5"><br><div class="gmail_quote">On Fri, Jul 20, 2012 at 4:35 PM, Alexey Samsonov <span dir="ltr"><<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi!<div><br></div><div>This patch fixes the representation of debug line table in DebugInfo LLVM library. Instead of just collection of rows, debug line table for compilation unit is now</div><div>treated as the number of row ranges, describing sequences [1]. The sequences are not always listed in the order of increasing address, so</div>



<div>the previously used std::lower_bound() sometimes produced wrong results. Now the instruction address lookup consists of two stages: finding the</div><div>correct sequence, and searching for address in range of rows for this sequence.</div>



<div><br></div><div>[1] dwarfstd: "Sequence - A series of contiguous target machine instructions. One compilation unit may emit multiple sequences</div><div>(that is, not all instructions within a compilation unit are assumed to be contiguous)"</div>



<div><br></div><div>Code review: <a href="http://codereview.appspot.com/6423064/" target="_blank">http://codereview.appspot.com/6423064/</a></div></blockquote></div><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>


</div></div></blockquote></div><br><br clear="all"><div><br></div>-- <br><div>Alexey Samsonov, MSK</div><br>
</div>