[Lldb-commits] [PATCH] D18381: Decouple ClangASTContext from DWARF

[Zachary Turner via lldb-commits]

zturner added a comment.

I guess my point was just that it seems a little weird to treat DWARF specially.  It's just another debug info format (albeit one that most compilers support), so by baking into the supposedly generic interface the requirement that "every type system must be able to support DWARF" seems a little strange.

I agree it solves the problem and everything will work, it just seems more correct from a design standpoint to have an enum for debug info type and expose one method called `GetDebugInfoParser` that takes the enum.


http://reviews.llvm.org/D18381