[PATCH] D32722: Verify that all references point to actual DIEs in "llvm-dwarfdump --verify"

Adrian Prantl via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 12:48:37 PDT 2017


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


================
Comment at: lib/DebugInfo/DWARF/DWARFContext.cpp:309
+    // lies between to valid DIEs.
+    std::map<uint64_t, std::set<uint32_t>> ReferenceToDIEOffsets;
+
----------------
Do we need a sorted container here or would an llvm::DenseMap be sufficient (and faster)?


https://reviews.llvm.org/D32722





More information about the llvm-commits mailing list