[PATCH] D38453: [dwarfdump] Verify that unit type matches root DIE

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Oct 9 11:07:33 PDT 2017


dblaikie added inline comments.


================
Comment at: llvm/trunk/include/llvm/BinaryFormat/Dwarf.h:348
+  case DW_TAG_skeleton_unit:
+  case DW_TAG_imported_unit:
+    return true;
----------------
This is probably incorrect - imported_units seem to be used to import a unit into another unit, they aren't themselves a valid top-level unit. (See Figure E.6 in DWARF5 for an example of their use)


Repository:
  rL LLVM

https://reviews.llvm.org/D38453





More information about the llvm-commits mailing list