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

[Paul Robinson via Phabricator via lldb-commits]

probinson added a comment.

I am looking at making the LLVM parser handle type units across v4/v5, and the trick of pretending .debug_types is pasted onto the end of .debug_info seems like a pretty convenient fiction.  It unifies DIE handling, and still allows the dumper to distinguish which bits are in which section, which is traditionally how the dumper UI thinks about DWARF.  So as long as you and Jan are on the same page there, I'm happy to follow suit.



================
Comment at: source/Plugins/SymbolFile/DWARF/DIERef.cpp:53
+      // and DIE for the type signature. When a type is referred to by a
+      // DW_FORM_ref_sig8 form, the real information for the type in
+      // contained in a DW_TAG_type_unit.
----------------
... the type is


https://reviews.llvm.org/D32167