[Lldb-commits] [Diffusion] rL245090: Move all clang type system DWARF type parsing into ClangASTContext.cpp.
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
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