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

[Jan Kratochvil via Phabricator via lldb-commits]

jankratochvil marked 5 inline comments as done.
jankratochvil added a comment.

In https://reviews.llvm.org/D47275#1110065, @clayborg wrote:

> Marked things that don't belong in DWARFBasicDIE.


OK, sorry, thanks for reviewing it.

> Also DWARFBasicDIE doesn't really explain what it actually is. Maybe we should rename this DWARFUnitDIE? DWARFTopDIE? DWARFRootDIE?

I do not mind any name but if you do not like Pavel's `DWARFBasicDIE` and you are not sure with the name yourself I have chosen `DWARFFirstDIE`.


https://reviews.llvm.org/D47275