[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