[PATCH] D32765: Add line table verification to lldb-dwarfdump --verify. AbandonedPublic

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue May 2 15:45:27 PDT 2017


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


================
Comment at: unittests/DebugInfo/DWARF/DWARFDebugInfoTest.cpp:1668-1671
+void VerifyError(DWARFContext &DwarfContext, StringRef Error) {
+  SmallString<1024> Str;
+  raw_svector_ostream Strm(Str);
+  EXPECT_FALSE(DwarfContext.verify(Strm, DIDT_All));
----------------
Presumably 'verify' should be const? (& then this parameter can be const as well)


https://reviews.llvm.org/D32765





More information about the llvm-commits mailing list