[PATCH] D44433: DWARFVerifier: Enhance validation of .debug_names hash tables

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added inline comments.


================
Comment at: lib/DebugInfo/DWARF/DWARFVerifier.cpp:848
+  }
+
+  for (uint32_t Bucket = 0, End = NI.getBucketCount(); Bucket < End; ++Bucket) {
----------------
A few more comments on what is being checked wouldn't hurt.


================
Comment at: lib/DebugInfo/DWARF/DWARFVerifier.cpp:881
+    if (B.Bucket == NI.getBucketCount())
+      break; // sentinel
+
----------------
Can you turn this comment into a full sentence?


Repository:
  rL LLVM

https://reviews.llvm.org/D44433