[PATCH] D35166: [DWARF] Introduce verification for the unit header chain in .debug_info section to llvm-dwarfdump.

[Spyridoula Gravani via Phabricator via llvm-commits]

sgravani added inline comments.


================
Comment at: lib/DebugInfo/DWARF/DWARFVerifier.cpp:103-104
+      Success = false;
+      if (isUnitDWARF64)
+        break;
+    }
----------------
friss wrote:
> I still don't like that isUnitDWARF64 is a member when it's used only for communicating between handleDebugInfoUnitHeaderChain() and verifyUnit(). I think I'd prefer is it was a local variable passed as an out-parameter to verifyUnit.
Changed initialization of isUnitDWARF64 to false so that the while loop that goes over units doesn't break after a failure in verifyUnitHeader().


https://reviews.llvm.org/D35166