[PATCH] D66472: [DWARF] Adjust return type of DWARFUnit::getLength().

Igor Kudrin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Aug 21 07:11:42 PDT 2019


This revision was automatically updated to reflect the committed changes.
Closed by commit rL369529: [DWARF] Adjust return type of DWARFUnit::getLength(). (authored by ikudrin, committed by ).

Changed prior to commit:
  https://reviews.llvm.org/D66472?vs=216118&id=216394#toc

Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D66472/new/

https://reviews.llvm.org/D66472

Files:
  llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnit.h
  llvm/trunk/lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
  llvm/trunk/lib/DebugInfo/DWARF/DWARFTypeUnit.cpp


Index: llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnit.h
===================================================================
--- llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnit.h
+++ llvm/trunk/include/llvm/DebugInfo/DWARF/DWARFUnit.h
@@ -286,7 +286,7 @@
   uint8_t getDwarfOffsetByteSize() const {
     return Header.getDwarfOffsetByteSize();
   }
-  uint32_t getLength() const { return Header.getLength(); }
+  uint64_t getLength() const { return Header.getLength(); }
   uint8_t getUnitType() const { return Header.getUnitType(); }
   bool isTypeUnit() const { return Header.isTypeUnit(); }
   uint64_t getNextUnitOffset() const { return Header.getNextUnitOffset(); }
Index: llvm/trunk/lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
===================================================================
--- llvm/trunk/lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
+++ llvm/trunk/lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
@@ -16,7 +16,7 @@
 
 void DWARFCompileUnit::dump(raw_ostream &OS, DIDumpOptions DumpOpts) {
   OS << format("0x%08" PRIx64, getOffset()) << ": Compile Unit:"
-     << " length = " << format("0x%08x", getLength())
+     << " length = " << format("0x%08" PRIx64, getLength())
      << " version = " << format("0x%04x", getVersion());
   if (getVersion() >= 5)
     OS << " unit_type = " << dwarf::UnitTypeString(getUnitType());
Index: llvm/trunk/lib/DebugInfo/DWARF/DWARFTypeUnit.cpp
===================================================================
--- llvm/trunk/lib/DebugInfo/DWARF/DWARFTypeUnit.cpp
+++ llvm/trunk/lib/DebugInfo/DWARF/DWARFTypeUnit.cpp
@@ -24,12 +24,12 @@
   if (DumpOpts.SummarizeTypes) {
     OS << "name = '" << Name << "'"
        << " type_signature = " << format("0x%016" PRIx64, getTypeHash())
-       << " length = " << format("0x%08x", getLength()) << '\n';
+       << " length = " << format("0x%08" PRIx64, getLength()) << '\n';
     return;
   }
 
   OS << format("0x%08" PRIx64, getOffset()) << ": Type Unit:"
-     << " length = " << format("0x%08x", getLength())
+     << " length = " << format("0x%08" PRIx64, getLength())
      << " version = " << format("0x%04x", getVersion());
   if (getVersion() >= 5)
     OS << " unit_type = " << dwarf::UnitTypeString(getUnitType());


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66472.216394.patch
Type: text/x-patch
Size: 2224 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190821/d4302b11/attachment.bin>


More information about the llvm-commits mailing list