[Lldb-commits] [PATCH] D47275: 1/3: DWARFDIE split out to DWARFBasicDIE

Pavel Labath via Phabricator via lldb-commits lldb-commits at lists.llvm.org
Thu May 24 03:49:33 PDT 2018


labath added a comment.

Looks like we missed each other, but all I said about DWARFUnitDIE applies to DWARFFirstDIE as well. I doesn't have to be called "basic" die, but the reason I proposed that name is that it does not sound weird when you say that any die "is a" basic die. Other possibility would be to keep DWARFDIE as the "basic" die, and have a something like a "navigatable" die for those that allow you to access children and stuff.


https://reviews.llvm.org/D47275





More information about the lldb-commits mailing list