[Lldb-commits] [PATCH] D62570: Use LLVM's debug line parser in LLDB
Adrian Prantl via Phabricator via lldb-commits
lldb-commits at lists.llvm.org
Tue Aug 13 10:00:40 PDT 2019
aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.
Two percent is a noticeable price tag, but since we are doing extra work with error handling and building a copy of the line table in memory first; understandable, I'm moderately optimistic that we could increase the performance a bit in the future by eliminating the extra copy step, though perhaps not with some larger changes to the LLVM interface. Since we need this for DWARF 5 support and the performance penalty isn't that bad, this patch gets a thumbs up from me. I ssume you profiled the code and there isn't any obvious silliness in the implementation?
CHANGES SINCE LAST ACTION
More information about the lldb-commits