[PATCH] D66472: [DWARF] Adjust return type of DWARFUnit::getLength().
Igor Kudrin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 20 06:05:43 PDT 2019
ikudrin created this revision.
ikudrin added reviewers: dblaikie, MaskRay, aprantl.
ikudrin added a project: LLVM.
`DWARFUnitHeader::getLenght()` returns `uint64_t`, so `DWARFUnit::getLength()` should do the same.
Repository:
rL LLVM
https://reviews.llvm.org/D66472
Files:
include/llvm/DebugInfo/DWARF/DWARFUnit.h
lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
lib/DebugInfo/DWARF/DWARFTypeUnit.cpp
Index: lib/DebugInfo/DWARF/DWARFTypeUnit.cpp
===================================================================
--- lib/DebugInfo/DWARF/DWARFTypeUnit.cpp
+++ 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());
Index: lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
===================================================================
--- lib/DebugInfo/DWARF/DWARFCompileUnit.cpp
+++ 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: include/llvm/DebugInfo/DWARF/DWARFUnit.h
===================================================================
--- include/llvm/DebugInfo/DWARF/DWARFUnit.h
+++ 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(); }
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D66472.216118.patch
Type: text/x-patch
Size: 2125 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20190820/5058ddbc/attachment.bin>
More information about the llvm-commits
mailing list