Author: Greg Clayton Date: 2026-01-28T17:49:18-08:00 New Revision: 16149ca93227d0d26a2f7387e95ad30971e99f43
URL: https://github.com/llvm/llvm-project/commit/16149ca93227d0d26a2f7387e95ad30971e99f43 DIFF: https://github.com/llvm/llvm-project/commit/16149ca93227d0d26a2f7387e95ad30971e99f43.diff LOG: Add support for DW_FORM_GNU_ref_alt and DW_FORM_GNU_strp_alt. (#176056) LLDB's DWARF parser didn't support parsing DW_FORM_GNU_ref_alt and DW_FORM_GNU_strp_alt forms which would cause any file loaded by LLDB to fail to parse any DWARF. Added support for parsing this information only, not for actually finding the debug info reference to an alternate file or a string in an alternate file. These extensions are used by DWZ files which are present in some linux distros, so it will be good for LLDB to just be able to parse these without emitting an error like: (lldb) b bar warning: (arm64) /tmp/a.out unsupported DW_FORM values: 0x1f20 0x1f21 Added: lldb/test/Shell/SymbolFile/DWARF/Inputs/gnu-ref-strp-alt.yaml lldb/test/Shell/SymbolFile/DWARF/gnu-ref-strp-alt.test Modified: lldb/source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp Removed: ################################################################################ diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp index dbb2a399dabff..2d53b8175f0a0 100644 --- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp +++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFFormValue.cpp @@ -76,6 +76,8 @@ bool DWARFFormValue::ExtractValue(const DWARFDataExtractor &data, case DW_FORM_strp: case DW_FORM_line_strp: case DW_FORM_sec_offset: + case DW_FORM_GNU_ref_alt: + case DW_FORM_GNU_strp_alt: assert(m_unit); m_value.uval = data.GetMaxU64( offset_ptr, m_unit->GetFormParams().getDwarfOffsetByteSize()); @@ -280,6 +282,8 @@ bool DWARFFormValue::SkipValue(dw_form_t form, case DW_FORM_sec_offset: case DW_FORM_strp: case DW_FORM_line_strp: + case DW_FORM_GNU_ref_alt: + case DW_FORM_GNU_strp_alt: assert(unit); *offset_ptr += unit->GetFormParams().getDwarfOffsetByteSize(); return true; @@ -416,6 +420,13 @@ void DWARFFormValue::Dump(Stream &s) const { m_unit->GetFormParams().getRefAddrByteSize()); break; } + case DW_FORM_GNU_ref_alt: + case DW_FORM_GNU_strp_alt: { + assert(m_unit); + DumpAddress(s.AsRawOstream(), uvalue, + m_unit->GetFormParams().getDwarfOffsetByteSize()); + break; + } case DW_FORM_ref1: unit_relative_offset = true; break; @@ -439,6 +450,7 @@ void DWARFFormValue::Dump(Stream &s) const { break; case DW_FORM_flag_present: break; + default: s.Printf("DW_FORM(0x%4.4x)", m_form); break; @@ -660,6 +672,8 @@ bool DWARFFormValue::FormIsSupported(dw_form_t form) { case DW_FORM_GNU_str_index: case DW_FORM_GNU_addr_index: case DW_FORM_implicit_const: + case DW_FORM_GNU_ref_alt: + case DW_FORM_GNU_strp_alt: return true; default: break; diff --git a/lldb/test/Shell/SymbolFile/DWARF/Inputs/gnu-ref-strp-alt.yaml b/lldb/test/Shell/SymbolFile/DWARF/Inputs/gnu-ref-strp-alt.yaml new file mode 100644 index 0000000000000..f4fb7d6d7c369 --- /dev/null +++ b/lldb/test/Shell/SymbolFile/DWARF/Inputs/gnu-ref-strp-alt.yaml @@ -0,0 +1,53 @@ +--- !ELF +FileHeader: + Class: ELFCLASS64 + Data: ELFDATA2LSB + Type: ET_DYN + Machine: EM_X86_64 + Entry: 0x1000 +DWARF: + debug_str: + - '' + - bar + debug_abbrev: + - ID: 0 + Table: + - Code: 0x1 + Tag: DW_TAG_compile_unit + Children: DW_CHILDREN_yes + Attributes: + - Attribute: DW_AT_name + Form: DW_FORM_GNU_strp_alt + - Attribute: DW_AT_language + Form: DW_FORM_udata + - Code: 0x2 + Tag: DW_TAG_subprogram + Children: DW_CHILDREN_no + Attributes: + - Attribute: DW_AT_name + Form: DW_FORM_strp + - Attribute: DW_AT_type + Form: DW_FORM_GNU_ref_alt + - Attribute: DW_AT_low_pc + Form: DW_FORM_addr + - Attribute: DW_AT_high_pc + Form: DW_FORM_addr + debug_info: + - Length: 0x27 + Version: 4 + AbbrevTableID: 0 + AbbrOffset: 0x0 + AddrSize: 8 + Entries: + - AbbrCode: 0x1 + Values: + - Value: 0x123 + - Value: 0x2 + - AbbrCode: 0x2 + Values: + - Value: 0x1 + - Value: 0x12 + - Value: 0x1000 + - Value: 0x1100 + - AbbrCode: 0x0 +... diff --git a/lldb/test/Shell/SymbolFile/DWARF/gnu-ref-strp-alt.test b/lldb/test/Shell/SymbolFile/DWARF/gnu-ref-strp-alt.test new file mode 100644 index 0000000000000..c81c41a8b5fab --- /dev/null +++ b/lldb/test/Shell/SymbolFile/DWARF/gnu-ref-strp-alt.test @@ -0,0 +1,12 @@ +# This test verifies that LLDB can read DWARF file that contains DW_FORM values +# encoded with the DW_FORM_GNU_ref_alt and DW_FORM_GNU_strp_alt. If any file was +# loaded into LLDB prior to adding support LLDB wouldn't be able to parse any +# of the DWARF as it wouldn't know how to parse these entries and since DWARF is +# a stream, nothing after that could be parsed. + +# RUN: yaml2obj %S/Inputs/gnu-ref-strp-alt.yaml > %t +# RUN: %lldb %t -b -o 'b bar' | FileCheck %s + +# CHECK: (lldb) b bar +# CHECK-NOT: unsupported DW_FORM values: +# CHECK: Breakpoint 1: address = 0x0000000000001000 _______________________________________________ lldb-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/lldb-commits
