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

Zachary Turner via lldb-commits lldb-commits at lists.llvm.org
Wed Mar 23 08:16:47 PDT 2016

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.


More information about the lldb-commits mailing list