I was interested in looking at refactoring it mostly as a software hygiene issue.
On Fri, Jun 20, 2014 at 5:36 PM, <[email protected]> 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 <[email protected]> wrote: > > > > On 20 June 2014 19:42, Zachary Turner <[email protected]> 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 > > [email protected] > > http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev > >
_______________________________________________ lldb-dev mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-dev
