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



_______________________________________________
lldb-commits mailing list
lldb-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits

Reply via email to