diff --git a/llvm/include/llvm/DebugInfo/DWARF/DWARFSection.h b/llvm/include/llvm/DebugInfo/DWARF/DWARFSection.h --- a/llvm/include/llvm/DebugInfo/DWARF/DWARFSection.h +++ b/llvm/include/llvm/DebugInfo/DWARF/DWARFSection.h @@ -15,6 +15,7 @@ struct DWARFSection { StringRef Data; + uint64_t Address = {}; }; struct SectionName { diff --git a/llvm/lib/DebugInfo/DWARF/DWARFContext.cpp b/llvm/lib/DebugInfo/DWARF/DWARFContext.cpp --- a/llvm/lib/DebugInfo/DWARF/DWARFContext.cpp +++ b/llvm/lib/DebugInfo/DWARF/DWARFContext.cpp @@ -823,6 +823,8 @@ if (DebugFrame) return DebugFrame.get(); + const DWARFSection &DS = DObj->getFrameSection(); + // There's a "bug" in the DWARFv3 standard with respect to the target address // size within debug frame sections. While DWARF is supposed to be independent // of its container, FDEs have fields with size being "target address size", @@ -832,9 +834,10 @@ // provides this information). This problem is fixed in DWARFv4 // See this dwarf-discuss discussion for more details: // http://lists.dwarfstd.org/htdig.cgi/dwarf-discuss-dwarfstd.org/2011-December/001173.html - DWARFDataExtractor debugFrameData(*DObj, DObj->getFrameSection(), - isLittleEndian(), DObj->getAddressSize()); - auto DF = std::make_unique(getArch(), /*IsEH=*/false); + DWARFDataExtractor debugFrameData(*DObj, DS, isLittleEndian(), + DObj->getAddressSize()); + auto DF = + std::make_unique(getArch(), /*IsEH=*/false, DS.Address); if (Error E = DF->parse(debugFrameData)) return std::move(E); @@ -846,10 +849,12 @@ if (EHFrame) return EHFrame.get(); - DWARFDataExtractor debugFrameData(*DObj, DObj->getEHFrameSection(), - isLittleEndian(), DObj->getAddressSize()); + const DWARFSection &DS = DObj->getEHFrameSection(); + DWARFDataExtractor debugFrameData(*DObj, DS, isLittleEndian(), + DObj->getAddressSize()); - auto DF = std::make_unique(getArch(), /*IsEH=*/true); + auto DF = + std::make_unique(getArch(), /*IsEH=*/true, DS.Address); if (Error E = DF->parse(debugFrameData)) return std::move(E); DebugFrame.swap(DF); @@ -1707,6 +1712,9 @@ if (Name == "debug_ranges") { // FIXME: Use the other dwo range section when we emit it. RangesDWOSection.Data = Data; + } else if (Name == "debug_frame" || Name == "eh_frame") { + if (DWARFSection *S = mapNameToDWARFSection(Name)) + S->Address = Section.getAddress(); } } else if (InfoSectionMap *Sections = StringSwitch(Name) diff --git a/llvm/test/tools/llvm-dwarfdump/X86/debug_frame_offset.test b/llvm/test/tools/llvm-dwarfdump/X86/debug_frame_offset.test --- a/llvm/test/tools/llvm-dwarfdump/X86/debug_frame_offset.test +++ b/llvm/test/tools/llvm-dwarfdump/X86/debug_frame_offset.test @@ -9,7 +9,7 @@ RUN: llvm-dwarfdump %p/../../dsymutil/Inputs/basic1.macho.x86_64.o \ RUN: -eh-frame=0x00000018 | FileCheck %s --check-prefix=EH EH: .eh_frame contents: -EH-NEXT: 00000018 00000024 0000001c FDE cie=00000000 pc=fffffffffffffd20...fffffffffffffd44 +EH-NEXT: 00000018 00000024 0000001c FDE cie=00000000 pc=00000000...00000024 EH-NEXT: Format: DWARF32 EH-NEXT: DW_CFA_advance_loc: 1 EH-NOT: pc diff --git a/llvm/test/tools/llvm-objdump/MachO/eh_frame-arm64.test b/llvm/test/tools/llvm-objdump/MachO/eh_frame-arm64.test --- a/llvm/test/tools/llvm-objdump/MachO/eh_frame-arm64.test +++ b/llvm/test/tools/llvm-objdump/MachO/eh_frame-arm64.test @@ -12,7 +12,7 @@ # CHECK: DW_CFA_def_cfa: reg31 +0 -# CHECK: 00000014 00000020 00000018 FDE cie=00000000 pc=00000000...00000020 +# CHECK: 00000014 00000020 00000018 FDE cie=00000000 pc=00000050...00000070 # CHECK: DW_CFA_advance_loc: 8 # CHECK: DW_CFA_def_cfa_offset: +16 # CHECK: DW_CFA_offset: reg30 -8