[Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.

via lldb-commits lldb-commits at lists.llvm.org
Thu Aug 27 14:08:38 PDT 2015


On Thu, Aug 27, 2015 at 12:56:57PM -0700, Greg Clayton via lldb-commits wrote:
> > 
> > Our Delphi compiler generates an alternate debug format as well - it took me all of yesterday to fix merge conflicts and I've still not got things working properly.  Greg, can you please move the DWARF dependencies back into the SymbolFileDWARF plugin?  Thanks!
> 
> Sorry to not be clear on this: thing are correctly organized and won't be changing.
> 
> You will need to merge your stuff correctly into the current world.

The issue is that you have added DWARF to ClangASTContext.  How is
another debug format plugin supposed to work in ClangASTContext now?

The design had been that support for debug formats goes in plugins
inside source/Plugins/SymbolFile.  This was a clean interface that
allowed support for other debug formats to exist by installing their
plugin.  Now main lldb code is polluted with DWARF code which depends on
a plugin.

Thanks,
-Dawn


More information about the lldb-commits mailing list