[lldb-dev] DWARF parsing code

[Zachary Turner]

I was interested in looking at refactoring it mostly as a software hygiene
issue.


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?
>
> Jim
>
> > 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
> forked
> >> from llvm\lib\DebugInfo.   Can anyone offer some history here?
> >>
> >> Is there any technical reason, aside from simply the work hasn't been
> done
> >> 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...
URL: <http://lists.llvm.org/pipermail/lldb-dev/attachments/20140620/6312a7e3/attachment.html>