[lldb-dev] DWARF parsing code
zturner at google.com
Fri Jun 20 17:38:10 PDT 2014
I was interested in looking at refactoring it mostly as a software hygiene
On Fri, Jun 20, 2014 at 5:36 PM, <jingham at apple.com> wrote:
> Note that lldb's needs for the DWARF parser are pretty specialized. It
> parses incrementally and puts a great deal of effort into not touching any
> more of the DWARF than it absolutely needs to to respond to the queries
> made to it. It is way overkill for a simple dwarf dumper. I'm a little
> leery of putting it into the llvm sources since somebody there who didn't
> understand all it was trying to do could make some seemingly innocent
> simplification, and that would not cause any correctness issues which the
> lldb testsuite would catch, but "only" performance problems that were
> probably only noticeable for sizable apps so the testsuite might not catch
> that either...
> Did you have some need for the lldb DWARF parser's capabilities or was
> this just a software-hygiene issue?
> > On Jun 20, 2014, at 5:23 PM, Ed Maste <emaste at freebsd.org> wrote:
> > On 20 June 2014 19:42, Zachary Turner <zturner at google.com> wrote:
> >> It seems to me like the code in source\Plugins\SymbolFile\DWARF was
> >> from llvm\lib\DebugInfo. Can anyone offer some history here?
> >> Is there any technical reason, aside from simply the work hasn't been
> >> yet, that we can't merge this back up into LLVM's DWARF parsing code and
> >> then just re-use that?
> > It's the other way around -- the code started out in LLDB, and was
> > then reused in LLVM, but nobody refactored LLDB to use the new one.
> > They have now diverged and each has some functionality that's not
> > present in the other one.
> > There's no technical reason we couldn't converge on one implementation
> > -- it's just a sizable effort that hasn't made it to the top of
> > anyone's priority list.
> > _______________________________________________
> > lldb-dev mailing list
> > lldb-dev at cs.uiuc.edu
> > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the lldb-dev