[Lldb-commits] [PATCH] D32167: Add support for type units (.debug_types) to LLDB in a way that is compatible with DWARF 5

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Mon Feb 26 13:00:43 PST 2018


labath added a comment.

In https://reviews.llvm.org/D32167#1019635, @jingham wrote:

> Note, I have (though very very occasionally) seen a dsymutil bug that broke line tables.  So it would be good to keep a few "set a breakpoint and run to it" tests just to sanity check this.  But most tests that just run somewhere and print a backtrace or view an integer variable or such-like do not need to run all variants.


The thing here is that I'm not sure everything can be captured by a single "debug info variant" dimension. The current dimensions kinda make sense, as in they alter the way pretty much all of debug info is accessed (we can think of "dwarf" as the basic one, "dsym" and "dwo" are similar in that they place everything in a separate file, I'm not sure about gmodules, but that's because I have no idea on how it works). However, I'm not sure the same goes for type units. For one they are not completely mutually exclusive with the existing variants, so you can have type-units+regular dwarf and type units+dwo (maybe even type units+dsym?). But obviously we can't add another "type unit" dimension and do a cartesian product....


https://reviews.llvm.org/D32167





More information about the lldb-commits mailing list