<div dir="ltr">This is definitely going to be a serious problem once I integrate our PDB support for Windows</div><br><div class="gmail_quote"><div dir="ltr">On Wed, Aug 26, 2015 at 9:09 PM Dawn Perchik 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">dawn added a subscriber: dawn.<br>
dawn added a comment.<br>
<br>
Hi Greg,<br>
<br>
While I really appreciate that you're making lldb less Clang specific (e.g. renaming ClangASTType to CompilerType), I'm concerned about this commit because it moves DWARF knowledge from the SymbolFileDWARF plugin into ClangASTContext.cpp, making it impossible to support other debug formats as plugins.<br>
<br>
I realize some DWARF had leaked outside of the DWARF plugin before this commit (like the enums for language and calling convention), but those could easily be converted to/from other debug formats.  This new code really requires that the debug info be DWARF.<br>
<br>
Am I missing something?<br>
Thanks in advance, -Dawn<br>
<br>
<br>
<a href="http://reviews.llvm.org/rL245090" rel="noreferrer" target="_blank">http://reviews.llvm.org/rL245090</a><br>
<br>
<br>
<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>