[PATCH] D46707: [DWARF] Factor out a DWARFUnitHeader class. NFC

[David Blaikie via Phabricator via llvm-commits]

dblaikie accepted this revision.
dblaikie added inline comments.
This revision is now accepted and ready to land.


================
Comment at: llvm/lib/DebugInfo/DWARF/DWARFUnit.cpp:114-116
+    // Fake a unit type based on the section type.  This isn't perfect,
+    // but to do better we'd need to look at the unit DIE.  The crucial
+    // distinction is type unit or not, because the header differs.
----------------
Not sure this comment makes sense - you can't look at the unit DIE, right? Because you have to parse the header before you parse the DIEs, so that doesn't seem like a relevant suggestion/approach here?


https://reviews.llvm.org/D46707