diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.h b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.h --- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.h +++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.h @@ -50,12 +50,13 @@ class DWARFDebugMacroEntry { public: - static void - ReadMacroEntries(const lldb_private::DWARFDataExtractor &debug_macro_data, - const lldb_private::DWARFDataExtractor &debug_str_data, - const bool offset_is_64_bit, lldb::offset_t *sect_offset, - SymbolFileDWARF *sym_file_dwarf, - lldb_private::DebugMacrosSP &debug_macros_sp); + static void ReadMacroEntries( + const lldb_private::DWARFDataExtractor &debug_macro_data, + const lldb_private::DWARFDataExtractor &debug_str_offset_data, + const lldb_private::DWARFDataExtractor &debug_str_data, + const bool offset_is_64_bit, lldb::offset_t *sect_offset, + SymbolFileDWARF *sym_file_dwarf, + lldb_private::DebugMacrosSP &debug_macros_sp); }; #endif // SymbolFileDWARF_DWARFDebugMacro_h_ diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.cpp --- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.cpp +++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugMacro.cpp @@ -58,6 +58,7 @@ void DWARFDebugMacroEntry::ReadMacroEntries( const DWARFDataExtractor &debug_macro_data, + const DWARFDataExtractor &debug_str_offset_data, const DWARFDataExtractor &debug_str_data, const bool offset_is_64_bit, lldb::offset_t *offset, SymbolFileDWARF *sym_file_dwarf, DebugMacrosSP &debug_macros_sp) { @@ -65,7 +66,9 @@ static_cast(debug_macro_data.GetU8(offset)); while (type != 0) { lldb::offset_t new_offset = 0, str_offset = 0; - uint32_t line = 0; + // For DWARF32 debug_str_offsets header size is 8 bytes. + uint8_t str_offsets_base = 8, index_size = 4; + uint32_t line = 0, str_index = 0; const char *macro_str = nullptr; uint32_t debug_line_file_idx = 0; @@ -96,6 +99,21 @@ debug_macros_sp->AddMacroEntry( DebugMacroEntry::CreateUndefEntry(line, macro_str)); break; + case DW_MACRO_define_strx: + case DW_MACRO_undef_strx: + line = debug_macro_data.GetULEB128(offset); + str_index = debug_macro_data.GetULEB128(offset); + str_offset = str_offsets_base + str_index * index_size; + str_offset = debug_str_offset_data.GetMaxU64(&str_offset, index_size); + macro_str = debug_str_data.GetCStr(&str_offset); + if (type == DW_MACRO_define_strx) + debug_macros_sp->AddMacroEntry( + DebugMacroEntry::CreateDefineEntry(line, macro_str)); + else + debug_macros_sp->AddMacroEntry( + DebugMacroEntry::CreateUndefEntry(line, macro_str)); + break; + case DW_MACRO_start_file: line = debug_macro_data.GetULEB128(offset); debug_line_file_idx = debug_macro_data.GetULEB128(offset); diff --git a/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp b/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp --- a/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp +++ b/lldb/source/Plugins/SymbolFile/DWARF/SymbolFileDWARF.cpp @@ -1056,8 +1056,9 @@ const DWARFDebugMacroHeader &header = DWARFDebugMacroHeader::ParseHeader(debug_macro_data, offset); DWARFDebugMacroEntry::ReadMacroEntries( - debug_macro_data, m_context.getOrLoadStrData(), header.OffsetIs64Bit(), - offset, this, debug_macros_sp); + debug_macro_data, m_context.getOrLoadStrOffsetsData(), + m_context.getOrLoadStrData(), header.OffsetIs64Bit(), offset, this, + debug_macros_sp); return debug_macros_sp; } diff --git a/lldb/test/Shell/Commands/Inputs/dwarf5-macro.s b/lldb/test/Shell/Commands/Inputs/dwarf5-macro.s new file mode 100644 --- /dev/null +++ b/lldb/test/Shell/Commands/Inputs/dwarf5-macro.s @@ -0,0 +1,266 @@ + .text + .file "test.c" + .file 0 "/home/" "test.c" md5 0xef6a7032e0c7ceeef621583f2c00dc80 + .globl main # -- Begin function main + .p2align 4, 0x90 + .type main,@function +main: # @main +.Lfunc_begin0: + .loc 0 9 0 # test.c:9:0 + .cfi_startproc +# %bb.0: + xorl %eax, %eax +.Ltmp0: + .loc 0 10 5 prologue_end # test.c:10:5 + movl $3, -4(%rsp) + .loc 0 11 5 # test.c:11:5 + movl $5, -8(%rsp) + .loc 0 12 1 # test.c:12:1 + retq +.Ltmp1: +.Lfunc_end0: + .size main, .Lfunc_end0-main + .cfi_endproc + # -- End function + .section .debug_abbrev,"",@progbits + .byte 1 # Abbreviation Code + .byte 17 # DW_TAG_compile_unit + .byte 1 # DW_CHILDREN_yes + .byte 37 # DW_AT_producer + .byte 37 # DW_FORM_strx1 + .byte 19 # DW_AT_language + .byte 5 # DW_FORM_data2 + .byte 3 # DW_AT_name + .byte 37 # DW_FORM_strx1 + .byte 114 # DW_AT_str_offsets_base + .byte 23 # DW_FORM_sec_offset + .byte 16 # DW_AT_stmt_list + .byte 23 # DW_FORM_sec_offset + .byte 27 # DW_AT_comp_dir + .byte 37 # DW_FORM_strx1 + .byte 17 # DW_AT_low_pc + .byte 27 # DW_FORM_addrx + .byte 18 # DW_AT_high_pc + .byte 6 # DW_FORM_data4 + .byte 115 # DW_AT_addr_base + .byte 23 # DW_FORM_sec_offset + .byte 121 # DW_AT_macros + .byte 23 # DW_FORM_sec_offset + .byte 0 # EOM(1) + .byte 0 # EOM(2) + .byte 2 # Abbreviation Code + .byte 46 # DW_TAG_subprogram + .byte 1 # DW_CHILDREN_yes + .byte 17 # DW_AT_low_pc + .byte 27 # DW_FORM_addrx + .byte 18 # DW_AT_high_pc + .byte 6 # DW_FORM_data4 + .byte 64 # DW_AT_frame_base + .byte 24 # DW_FORM_exprloc + .byte 3 # DW_AT_name + .byte 37 # DW_FORM_strx1 + .byte 58 # DW_AT_decl_file + .byte 11 # DW_FORM_data1 + .byte 59 # DW_AT_decl_line + .byte 11 # DW_FORM_data1 + .byte 73 # DW_AT_type + .byte 19 # DW_FORM_ref4 + .byte 63 # DW_AT_external + .byte 25 # DW_FORM_flag_present + .byte 0 # EOM(1) + .byte 0 # EOM(2) + .byte 3 # Abbreviation Code + .byte 52 # DW_TAG_variable + .byte 0 # DW_CHILDREN_no + .byte 2 # DW_AT_location + .byte 24 # DW_FORM_exprloc + .byte 3 # DW_AT_name + .byte 37 # DW_FORM_strx1 + .byte 58 # DW_AT_decl_file + .byte 11 # DW_FORM_data1 + .byte 59 # DW_AT_decl_line + .byte 11 # DW_FORM_data1 + .byte 73 # DW_AT_type + .byte 19 # DW_FORM_ref4 + .byte 0 # EOM(1) + .byte 0 # EOM(2) + .byte 4 # Abbreviation Code + .byte 36 # DW_TAG_base_type + .byte 0 # DW_CHILDREN_no + .byte 3 # DW_AT_name + .byte 37 # DW_FORM_strx1 + .byte 62 # DW_AT_encoding + .byte 11 # DW_FORM_data1 + .byte 11 # DW_AT_byte_size + .byte 11 # DW_FORM_data1 + .byte 0 # EOM(1) + .byte 0 # EOM(2) + .byte 0 # EOM(3) + .section .debug_info,"",@progbits +.Lcu_begin0: + .long .Ldebug_info_end0-.Ldebug_info_start0 # Length of Unit +.Ldebug_info_start0: + .short 5 # DWARF version number + .byte 1 # DWARF Unit Type + .byte 8 # Address Size (in bytes) + .long .debug_abbrev # Offset Into Abbrev. Section + .byte 1 # Abbrev [1] 0xc:0x46 DW_TAG_compile_unit + .byte 0 # DW_AT_producer + .short 12 # DW_AT_language + .byte 1 # DW_AT_name + .long .Lstr_offsets_base0 # DW_AT_str_offsets_base + .long .Lline_table_start0 # DW_AT_stmt_list + .byte 2 # DW_AT_comp_dir + .byte 0 # DW_AT_low_pc + .long .Lfunc_end0-.Lfunc_begin0 # DW_AT_high_pc + .long .Laddr_table_base0 # DW_AT_addr_base + .long .Lcu_macro_begin0-.debug_macro # DW_AT_macros + .byte 2 # Abbrev [2] 0x27:0x26 DW_TAG_subprogram + .byte 0 # DW_AT_low_pc + .long .Lfunc_end0-.Lfunc_begin0 # DW_AT_high_pc + .byte 1 # DW_AT_frame_base + .byte 87 + .byte 3 # DW_AT_name + .byte 0 # DW_AT_decl_file + .byte 9 # DW_AT_decl_line + .long 77 # DW_AT_type + # DW_AT_external + .byte 3 # Abbrev [3] 0x36:0xb DW_TAG_variable + .byte 2 # DW_AT_location + .byte 145 + .byte 124 + .byte 5 # DW_AT_name + .byte 0 # DW_AT_decl_file + .byte 10 # DW_AT_decl_line + .long 77 # DW_AT_type + .byte 3 # Abbrev [3] 0x41:0xb DW_TAG_variable + .byte 2 # DW_AT_location + .byte 145 + .byte 120 + .byte 6 # DW_AT_name + .byte 0 # DW_AT_decl_file + .byte 11 # DW_AT_decl_line + .long 77 # DW_AT_type + .byte 0 # End Of Children Mark + .byte 4 # Abbrev [4] 0x4d:0x4 DW_TAG_base_type + .byte 4 # DW_AT_name + .byte 5 # DW_AT_encoding + .byte 4 # DW_AT_byte_size + .byte 0 # End Of Children Mark +.Ldebug_info_end0: + .section .debug_macro,"",@progbits +.Lcu_macro_begin0: + .short 5 # Macro information version + .byte 2 # Flags: 32 bit, lineptr present + .long .Lline_table_start0 # lineptr + .byte 3 # DW_MACRO_start_file + .byte 0 # Line Number + .byte 0 # File Number + .byte 3 # DW_MACRO_start_file + .byte 1 # Line Number + .file 1 "." "foo.h" md5 0x0f0cd0e22b44f49d3944992c8dc28661 # File Number + .byte 1 + .byte 11 # DW_MACRO_define_strx + .byte 1 # Line Number + .byte 7 # Macro String Index + .byte 4 # DW_MACRO_end_file + .byte 3 # DW_MACRO_start_file + .byte 2 # Line Number + .file 2 "." "bar.h" md5 0xbf4b34c333eaaa1d7085c25313b8d100 # File Number + .byte 2 + .byte 11 # DW_MACRO_define_strx + .byte 1 # Line Number + .byte 8 # Macro String Index + .byte 4 # DW_MACRO_end_file + .byte 11 # DW_MACRO_define_strx + .byte 4 # Line Number + .byte 9 # Macro String Index + .byte 11 # DW_MACRO_define_strx + .byte 5 # Line Number + .byte 10 # Macro String Index + .byte 12 # DW_MACRO_undef_strx + .byte 14 # Line Number + .byte 11 # Macro String Index + .byte 4 # DW_MACRO_end_file + .byte 11 # DW_MACRO_define_strx + .byte 0 # Line Number + .byte 12 # Macro String Index + .byte 11 # DW_MACRO_define_strx + .byte 0 # Line Number + .byte 13 # Macro String Index + .byte 11 # DW_MACRO_define_strx + .byte 0 # Line Number + .byte 14 # Macro String Index + .byte 11 # DW_MACRO_define_strx + .byte 0 # Line Number + .byte 15 # Macro String Index + .byte 0 # End Of Macro List Mark + .section .debug_str_offsets,"",@progbits + .long 68 + .short 5 + .short 0 +.Lstr_offsets_base0: + .section .debug_str,"MS",@progbits,1 +.Linfo_string0: + .asciz "clang version 10.0.0" # string offset=0 +.Linfo_string1: + .asciz "test.c" # string offset=21 +.Linfo_string2: + .asciz "/home/" # string offset=28 +.Linfo_string3: + .asciz "main" # string offset=35 +.Linfo_string4: + .asciz "int" # string offset=40 +.Linfo_string5: + .asciz "a" # string offset=44 +.Linfo_string6: + .asciz "b" # string offset=46 +.Linfo_string7: + .asciz "FOO 5" # string offset=48 +.Linfo_string8: + .asciz "BAR 6" # string offset=54 +.Linfo_string9: + .asciz "BAZ 7" # string offset=60 +.Linfo_string10: + .asciz "YEA 8" # string offset=66 +.Linfo_string11: + .asciz "YEA " # string offset=72 +.Linfo_string12: + .asciz "__llvm__ 1" # string offset=77 +.Linfo_string13: + .asciz "__clang__ 1" # string offset=88 +.Linfo_string14: + .asciz "__clang_major__ 10" # string offset=100 +.Linfo_string15: + .asciz "__clang_minor__ 0" # string offset=119 + .section .debug_str_offsets,"",@progbits + .long .Linfo_string0 + .long .Linfo_string1 + .long .Linfo_string2 + .long .Linfo_string3 + .long .Linfo_string4 + .long .Linfo_string5 + .long .Linfo_string6 + .long .Linfo_string7 + .long .Linfo_string8 + .long .Linfo_string9 + .long .Linfo_string10 + .long .Linfo_string11 + .long .Linfo_string12 + .long .Linfo_string13 + .long .Linfo_string14 + .long .Linfo_string15 + .section .debug_addr,"",@progbits + .long .Ldebug_addr_end0-.Ldebug_addr_start0 # Length of contribution +.Ldebug_addr_start0: + .short 5 # DWARF version number + .byte 8 # Address size + .byte 0 # Segment selector size +.Laddr_table_base0: + .quad .Lfunc_begin0 +.Ldebug_addr_end0: + .ident "clang version 10.0.0" + .section ".note.GNU-stack","",@progbits + .addrsig + .section .debug_line,"",@progbits +.Lline_table_start0: diff --git a/lldb/test/Shell/Commands/dwarf5-macro.test b/lldb/test/Shell/Commands/dwarf5-macro.test new file mode 100644 --- /dev/null +++ b/lldb/test/Shell/Commands/dwarf5-macro.test @@ -0,0 +1,16 @@ +# REQUIRES: x86 +# This test checks lldb macro expansion when macro section +# contains DWARFv5 DW_MACRO_define_strx/DW_MACRO_undef_strx forms +# +# RUN: %clang_host -gdwarf-5 %p/Inputs/dwarf5-macro.s -o %t +# RUN: %lldb -s %s %t 2>&1 | FileCheck %s + +display FOO +display BAR +b main +run +# CHECK: Hook {{.*}} (expr -- FOO) +# CHECK-NEXT: {{.*}} 5 +# +# CHECK: Hook {{.*}} (expr -- BAR) +# CHECK-NEXT: {{.*}} 6