[llvm] Aggregate errors from llvm-dwarfutil --verify (PR #79648)

David Blaikie via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 26 15:34:17 PST 2024


================
@@ -348,6 +363,9 @@ class DWARFVerifier {
   bool verifyDebugStrOffsets(
       StringRef SectionName, const DWARFSection &Section, StringRef StrData,
       void (DWARFObject::*)(function_ref<void(const DWARFSection &)>) const);
+
+  /// Emits any aggregate information collected, depending on the dump options
+  void summarize(bool Success);
----------------
dwblaikie wrote:

If we have this aggregator that gets called back on every error anyway - would that allow us to remove the `Success` returns and tracking, and just track success in the aggregator instead?

https://github.com/llvm/llvm-project/pull/79648


More information about the llvm-commits mailing list