[PATCH] D51081: [DWARF v5] Refactoring range list dumping to fold DWARF v4 functionality into v5 handling (almost NFC).

[Wolfgang Pieb via Phabricator via llvm-commits]

wolfgangp added inline comments.


================
Comment at: test/DebugInfo/Generic/cu-ranges.ll:17-20
+; CHECK-NEXT: 0x00000000: 0x0000000000000000, 0x0000000000000011 =>
+; CHECK-SAME: [0x0000000000000000, 0x0000000000000011)
+; CHECK-NEXT: 0x00000010: 0x0000000000000000, 0x0000000000000011 =>
+; CHECK-SAME: [0x0000000000000000, 0x0000000000000011)
----------------
dblaikie wrote:
> Not sure if this is necessary/improving the check, perhaps? (if this test was going to check for the address ranges - it could've been doing that already, I'm not sure the recent changes to dumping make this more viable/appropriate? Not sure though)
Perhaps this change would capture the intent of the test better, if we want to verify that ranges for 2 different sections are created, i.e. more relying on the ability to display the section.


https://reviews.llvm.org/D51081