From 63eca15e95adb8e4049cd5b88eeaa37bd9175b6a Mon Sep 17 00:00:00 2001 From: Jonas Devlieghere <jonas@devlieghere.com> Date: Tue, 22 May 2018 17:38:03 +0000 Subject: [PATCH] [DebugInfo] Invert DIE order for range errors. When printing an error for an invalid address range in a DIE, we used to print the child above the parent, which is counter intuitive. This patch reverses the order and indents the child to mimic the way we print the debug info section. llvm-svn: 333006 --- llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp index 356b141f44fe..67da9ccaefd2 100644 --- a/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp +++ b/llvm/lib/DebugInfo/DWARF/DWARFVerifier.cpp @@ -362,10 +362,9 @@ unsigned DWARFVerifier::verifyDieRanges(const DWARFDie &Die, ParentRI.Die.getTag() == DW_TAG_subprogram); if (ShouldBeContained && !ParentRI.contains(RI)) { ++NumErrors; - error() << "DIE address ranges are not " - "contained in its parent's ranges:"; - Die.dump(OS, 0); + error() << "DIE address ranges are not contained in its parent's ranges:"; ParentRI.Die.dump(OS, 0); + Die.dump(OS, 2); OS << "\n"; } -- GitLab