[llvm] r260619 - [codeview] Dump def range lengths in hex

[Reid Kleckner via llvm-commits]

Author: rnk
Date: Thu Feb 11 17:40:14 2016
New Revision: 260619

URL: http://llvm.org/viewvc/llvm-project?rev=260619&view=rev
Log:
[codeview] Dump def range lengths in hex

It makes it easier to correlate with assembly dumps, which are typically
given with hex offsets.

Modified:
    llvm/trunk/test/DebugInfo/COFF/local-variables.ll
    llvm/trunk/test/MC/COFF/cv-def-range.s
    llvm/trunk/tools/llvm-readobj/COFFDumper.cpp

Modified: llvm/trunk/test/DebugInfo/COFF/local-variables.ll
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/DebugInfo/COFF/local-variables.ll?rev=260619&r1=260618&r2=260619&view=diff
==============================================================================
--- llvm/trunk/test/DebugInfo/COFF/local-variables.ll (original)
+++ llvm/trunk/test/DebugInfo/COFF/local-variables.ll Thu Feb 11 17:40:14 2016
@@ -122,7 +122,7 @@
 ; OBJ:      LocalVariableAddrRange {
 ; OBJ:        OffsetStart: .text+0x8
 ; OBJ:        ISectStart: 0x0
-; OBJ:        Range: 79
+; OBJ:        Range: 0x4F
 ; OBJ:      }
 ; OBJ:    }
 ; OBJ:    Local {
@@ -139,7 +139,7 @@
 ; OBJ:      LocalVariableAddrRange {
 ; OBJ:        OffsetStart: .text+0xC
 ; OBJ:        ISectStart: 0x0
-; OBJ:        Range: 33
+; OBJ:        Range: 0x21
 ; OBJ:      }
 ; OBJ:    }
 ; OBJ:    Local {
@@ -156,7 +156,7 @@
 ; OBJ:      LocalVariableAddrRange {
 ; OBJ:        OffsetStart: .text+0x2D
 ; OBJ:        ISectStart: 0x0
-; OBJ:        Range: 36
+; OBJ:        Range: 0x24
 ; OBJ:      }
 ; OBJ:    }
 ; OBJ:    InlineSite {
@@ -184,7 +184,7 @@
 ; OBJ:      LocalVariableAddrRange {
 ; OBJ:        OffsetStart: .text+0x14
 ; OBJ:        ISectStart: 0x0
-; OBJ:        Range: 25
+; OBJ:        Range: 0x19
 ; OBJ:      }
 ; OBJ:    }
 ; OBJ:    InlineSiteEnd {
@@ -214,7 +214,7 @@
 ; OBJ:      LocalVariableAddrRange {
 ; OBJ:        OffsetStart: .text+0x35
 ; OBJ:        ISectStart: 0x0
-; OBJ:        Range: 23
+; OBJ:        Range: 0x17
 ; OBJ:      }
 ; OBJ:    }
 ; OBJ:    InlineSiteEnd {

Modified: llvm/trunk/test/MC/COFF/cv-def-range.s
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/test/MC/COFF/cv-def-range.s?rev=260619&r1=260618&r2=260619&view=diff
==============================================================================
--- llvm/trunk/test/MC/COFF/cv-def-range.s (original)
+++ llvm/trunk/test/MC/COFF/cv-def-range.s Thu Feb 11 17:40:14 2016
@@ -81,7 +81,7 @@ Ltmp3:
 # CHECK:      LocalVariableAddrRange {
 # CHECK:        OffsetStart: .text+0x9
 # CHECK:        ISectStart: 0x0
-# CHECK:        Range: 15
+# CHECK:        Range: 0xF
 # CHECK:      }
 # CHECK:    }
 # CHECK:    BlockRelocations [

Modified: llvm/trunk/tools/llvm-readobj/COFFDumper.cpp
URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/tools/llvm-readobj/COFFDumper.cpp?rev=260619&r1=260618&r2=260619&view=diff
==============================================================================
--- llvm/trunk/tools/llvm-readobj/COFFDumper.cpp (original)
+++ llvm/trunk/tools/llvm-readobj/COFFDumper.cpp Thu Feb 11 17:40:14 2016
@@ -1907,7 +1907,7 @@ void COFFDumper::printLocalVariableAddrR
   DictScope S(W, "LocalVariableAddrRange");
   printRelocatedField("OffsetStart", Sec, SectionContents, &Range.OffsetStart);
   W.printHex("ISectStart", Range.ISectStart);
-  W.printNumber("Range", Range.Range);
+  W.printHex("Range", Range.Range);
 }
 
 void COFFDumper::printLocalVariableAddrGap(StringRef &SymData) {
@@ -1915,8 +1915,8 @@ void COFFDumper::printLocalVariableAddrG
     const LocalVariableAddrGap *Gap;
     error(consumeObject(SymData, Gap));
     ListScope S(W, "LocalVariableAddrGap");
-    W.printNumber("GapStartOffset", Gap->GapStartOffset);
-    W.printNumber("Range", Gap->Range);
+    W.printHex("GapStartOffset", Gap->GapStartOffset);
+    W.printHex("Range", Gap->Range);
   }
 }