<div dir="ltr">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?<div><br></div><div>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.</div></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Aug 27, 2015 at 2:08 PM via lldb-commits <<a href="mailto:lldb-commits@lists.llvm.org">lldb-commits@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Thu, Aug 27, 2015 at 12:56:57PM -0700, Greg Clayton via lldb-commits wrote:<br>
> ><br>
> > 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!<br>
><br>
> Sorry to not be clear on this: thing are correctly organized and won't be changing.<br>
><br>
> You will need to merge your stuff correctly into the current world.<br>
<br>
The issue is that you have added DWARF to ClangASTContext.  How is<br>
another debug format plugin supposed to work in ClangASTContext now?<br>
<br>
The design had been that support for debug formats goes in plugins<br>
inside source/Plugins/SymbolFile.  This was a clean interface that<br>
allowed support for other debug formats to exist by installing their<br>
plugin.  Now main lldb code is polluted with DWARF code which depends on<br>
a plugin.<br>
<br>
Thanks,<br>
-Dawn<br>
_______________________________________________<br>
lldb-commits mailing list<br>
<a href="mailto:lldb-commits@lists.llvm.org" target="_blank">lldb-commits@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits</a><br>
</blockquote></div>