<div dir="ltr">Suppose you've got two line sequences.  <div><br></div><div>First sequence:</div><div>4198512</div><div>4198544</div><div>4198547</div><div>4198562</div><div><br></div><div>Second sequence:</div><div>4198528</div><div>4198531</div><div>4198537</div><div>4198552</div><div><br></div><div>These two line sequences overlap, and will not be inserted correctly into a LineTable.  This sounds like a bug to me, because as far as I can tell there is nothing preventing LineSequences being organized this way, but LineTable::InsertSequence assumes that this cannot happen.</div></div><br><div class="gmail_quote"><div dir="ltr">On Fri, Feb 12, 2016 at 11:01 AM Zachary Turner <<a href="mailto:zturner@google.com">zturner@google.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="gmail_quote"><div dir="ltr">On Thu, Feb 11, 2016 at 5:35 PM Greg Clayton <<a href="mailto:gclayton@apple.com" target="_blank">gclayton@apple.com</a>> wrote:<br></div></div></div><div dir="ltr"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><br>
> 5. ParseCompileUnitLineTable.  On the LineTable class you can add "line sequences" or individual entries.  What's the difference here?  Is there any disadvantage to adding every single line entry in the line table using the InsertLineEntry instead of building a line sequence and inserting the sequence?<br>
<br>
The rule follows DWARF line tables: line sequences must be an array of line entries whose addresses are always increasing. You can add every line in sequence as long as the line entries are in increasing address order. We are going to sort the line entries into an array that is sorted for quick lookups.<br></blockquote></div></div><div dir="ltr"><div class="gmail_quote"><div>Just to make sure I understand, semantically here, there is nothing special about a line sequence, it's just an optimization to let LineTable know you're giving it sorted values?  So any lines you add via a LineSequence, could also be added individually with insertLine, but it would be slower?  And aside from that everything else would still work as expected?</div></div></div></blockquote></div>