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!  :)
_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to