[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:18:09 PDT 2015

On Thu, Aug 27, 2015 at 09:12:06PM +0000, Zachary Turner wrote:
> If I'm following this discussion correctly, Greg are you saying that
> ClangASTContext could be renamed to DwarfASTContext?  And that for other
> types of Debug info we could have PascalASTContext and PdbASTContext?
> If that's the case, then it seems there is a lot of code inside of
> ClangASTContext (aka DwarfASTContext) which is not specific to DWARF, and
> there's no reason that other ASTContext classes shouldn't be able to reuse
> it.

Renaming and reusing the ClangASTContext code would help but it's not
enough - the DWARF code belongs in the DWARF plugin.  Plugins should be
just that - pluggable!  :)

More information about the lldb-commits mailing list