[Lldb-commits] [PATCH] D72909: Make SymbolFileDWARF::ParseLineTable use std::sort instead of insertion sort

Unnar Freyr Erlendsson via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Fri Jan 17 03:23:48 PST 2020


unnar added a comment.

In D72909#1826110 <https://reviews.llvm.org/D72909#1826110>, @labath wrote:

> Thanks for the patch. I've been wondering how to improve this, and this solution is pretty neat.
>
> I have just one request. Instead of the `ReplaceLineTableWithSequences` thingy, could you just create a LineTable constructor, which takes an `ArrayRef<LineSequence *>` or similar. It looks like all it takes is to make `CreateLineSequenceContainer` static, so you can invoke it without a `LineTable` instance. (There are other possible cleanups here too, but I don't think its your job do fix all of those...)


Thanks for the quick review!

It also requires AppendLineEntryToSequence to be static but I assume that should also be just fine.


Repository:
  rLLDB LLDB

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D72909/new/

https://reviews.llvm.org/D72909





More information about the lldb-commits mailing list