[PATCH] [dwarfdump] Dump DW_AT_ranges values inline in the debug_info dump.

[David Blaikie]

Oops, sorry, forgot to mash the 'send' button...

================
Comment at: lib/DebugInfo/DWARFDebugInfoEntry.cpp:154
@@ -134,1 +153,3 @@
         OS << " \"" << Ref << '\"';
+  } else if (attr == DW_AT_ranges) {
+    dumpRanges(OS, getAddressRanges(u), u->getAddressByteSize(),
----------------
What's the failure mode if DW_AT_ranges isn't an index, or is not a correct index? (outside the range of ranges, etc)

================
Comment at: test/DebugInfo/dwarfdump-ranges.test:16
@@ +15,3 @@
+CHECK-NEXT:          [0x0000000000000637 - 0x000000000000063d)
+CHECK-NEXT:          End)
+
----------------
I'd probably skip the "End" word and just put ")" on the end of the previous line - given the consistent ending, line length, etc, it should be pretty obvious.

But I'm open to other ideas. 

What does Darwin's dwarfdump formatting for this look like?

http://reviews.llvm.org/D5712