<div>Could you please take a look?</div><div><br></div><div>(re-attaching the patch just in case)</div><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>