[Lldb-commits] [lldb] 24f0753 - [lldb] Fix DW_AT_decl_file from DW_AT_specification from a different CU
Jan Kratochvil via lldb-commits
lldb-commits at lists.llvm.org
Mon Nov 9 01:53:14 PST 2020
Author: Jan Kratochvil
Date: 2020-11-09T10:52:48+01:00
New Revision: 24f07531a34e141257f6616611e6430aed6f60a1
URL: https://github.com/llvm/llvm-project/commit/24f07531a34e141257f6616611e6430aed6f60a1
DIFF: https://github.com/llvm/llvm-project/commit/24f07531a34e141257f6616611e6430aed6f60a1.diff
LOG: [lldb] Fix DW_AT_decl_file from DW_AT_specification from a different CU
This would be reproducible in future DWZ category of the testsuite as:
Failed Tests (1):
lldb-api :: python_api/symbol-context/two-files/TestSymbolContextTwoFiles.py
Differential Revision: https://reviews.llvm.org/D91014
Added:
lldb/test/Shell/SymbolFile/DWARF/DW_AT_decl_file-DW_AT_specification-crosscu.s
Modified:
lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp
lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.cpp
lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.h
lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp
lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.h
Removed:
################################################################################
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp
index 2fee87d98da8..2003a24c04fa 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp
+++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFASTParserClang.cpp
@@ -341,7 +341,9 @@ ParsedDWARFTypeAttributes::ParsedDWARFTypeAttributes(const DWARFDIE &die) {
break;
case DW_AT_decl_file:
- decl.SetFile(die.GetCU()->GetFile(form_value.Unsigned()));
+ // die.GetCU() can
diff er if DW_AT_specification uses DW_FORM_ref_addr.
+ decl.SetFile(
+ attributes.CompileUnitAtIndex(i)->GetFile(form_value.Unsigned()));
break;
case DW_AT_decl_line:
decl.SetLine(form_value.Unsigned());
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.cpp
index 6c72d9e26221..ef98d7e33a90 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.cpp
+++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.cpp
@@ -25,7 +25,7 @@ uint32_t DWARFAttributes::FindAttributeIndex(dw_attr_t attr) const {
return UINT32_MAX;
}
-void DWARFAttributes::Append(const DWARFUnit *cu, dw_offset_t attr_die_offset,
+void DWARFAttributes::Append(DWARFUnit *cu, dw_offset_t attr_die_offset,
dw_attr_t attr, dw_form_t form) {
AttributeValue attr_value = {
cu, attr_die_offset, {attr, form, DWARFFormValue::ValueType()}};
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.h b/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.h
index 9948969f108f..8c21404610d7 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.h
+++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFAttribute.h
@@ -52,11 +52,9 @@ class DWARFAttributes {
DWARFAttributes();
~DWARFAttributes();
- void Append(const DWARFUnit *cu, dw_offset_t attr_die_offset,
- dw_attr_t attr, dw_form_t form);
- const DWARFUnit *CompileUnitAtIndex(uint32_t i) const {
- return m_infos[i].cu;
- }
+ void Append(DWARFUnit *cu, dw_offset_t attr_die_offset, dw_attr_t attr,
+ dw_form_t form);
+ DWARFUnit *CompileUnitAtIndex(uint32_t i) const { return m_infos[i].cu; }
dw_offset_t DIEOffsetAtIndex(uint32_t i) const {
return m_infos[i].die_offset;
}
@@ -73,8 +71,8 @@ class DWARFAttributes {
protected:
struct AttributeValue {
- const DWARFUnit *cu; // Keep the compile unit with each attribute in
- // case we have DW_FORM_ref_addr values
+ DWARFUnit *cu; // Keep the compile unit with each attribute in
+ // case we have DW_FORM_ref_addr values
dw_offset_t die_offset;
DWARFAttribute attr;
};
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp
index f6425a889da8..3eca911f4837 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp
+++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.cpp
@@ -399,7 +399,7 @@ bool DWARFDebugInfoEntry::GetDIENamesAndRanges(
// specification or abstract origin attributes and including those in the
// results. Any duplicate attributes will have the first instance take
// precedence (this can happen for declaration attributes).
-size_t DWARFDebugInfoEntry::GetAttributes(const DWARFUnit *cu,
+size_t DWARFDebugInfoEntry::GetAttributes(DWARFUnit *cu,
DWARFAttributes &attributes,
Recurse recurse,
uint32_t curr_depth) const {
diff --git a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.h b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.h
index 3019e1767f18..534d92e1feb9 100644
--- a/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.h
+++ b/lldb/source/Plugins/SymbolFile/DWARF/DWARFDebugInfoEntry.h
@@ -49,7 +49,7 @@ class DWARFDebugInfoEntry {
const DWARFUnit *cu, lldb::offset_t *offset_ptr);
using Recurse = DWARFBaseDIE::Recurse;
- size_t GetAttributes(const DWARFUnit *cu, DWARFAttributes &attrs,
+ size_t GetAttributes(DWARFUnit *cu, DWARFAttributes &attrs,
Recurse recurse = Recurse::yes) const {
return GetAttributes(cu, attrs, recurse, 0 /* curr_depth */);
}
@@ -180,8 +180,8 @@ class DWARFDebugInfoEntry {
dw_tag_t m_tag = llvm::dwarf::DW_TAG_null;
private:
- size_t GetAttributes(const DWARFUnit *cu, DWARFAttributes &attrs,
- Recurse recurse, uint32_t curr_depth) const;
+ size_t GetAttributes(DWARFUnit *cu, DWARFAttributes &attrs, Recurse recurse,
+ uint32_t curr_depth) const;
};
#endif // LLDB_SOURCE_PLUGINS_SYMBOLFILE_DWARF_DWARFDEBUGINFOENTRY_H
diff --git a/lldb/test/Shell/SymbolFile/DWARF/DW_AT_decl_file-DW_AT_specification-crosscu.s b/lldb/test/Shell/SymbolFile/DWARF/DW_AT_decl_file-DW_AT_specification-crosscu.s
new file mode 100644
index 000000000000..c8c85cbc42df
--- /dev/null
+++ b/lldb/test/Shell/SymbolFile/DWARF/DW_AT_decl_file-DW_AT_specification-crosscu.s
@@ -0,0 +1,202 @@
+# Check that DW_AT_decl_file inherited by DW_AT_specification from a
diff erent
+# DW_TAG_compile_unit is using the DW_TAG_compile_unit->DW_AT_stmt_list where the
+# DW_AT_decl_file is located (and not where the DW_AT_specification is located).
+
+# REQUIRES: x86
+
+# RUN: llvm-mc -filetype=obj -triple x86_64-pc-linux %s -o %t
+# RUN: %lldb %t \
+# RUN: -o "image lookup -v -n main" \
+# RUN: -o exit | FileCheck %s
+
+# CHECK: FuncType: id = {0x0000002a}, byte-size = 0, decl = filename1:1, compiler_type = "int (void)"
+
+ .text
+ .globl main # -- Begin function main
+ .type main, at function
+main: # @main
+.Lfunc_begin0:
+ .byte 0
+.Lfunc_end0:
+ .size main, .Lfunc_end0-main
+ # -- End function
+ .section .debug_abbrev,"", at progbits
+ .byte 1 # Abbreviation Code
+ .byte 17 # DW_TAG_compile_unit
+ .byte 1 # DW_CHILDREN_yes
+ .byte 37 # DW_AT_producer
+ .byte 14 # DW_FORM_strp
+ .byte 19 # DW_AT_language
+ .byte 5 # DW_FORM_data2
+ .byte 3 # DW_AT_name
+ .byte 14 # DW_FORM_strp
+ .byte 16 # DW_AT_stmt_list
+ .byte 23 # DW_FORM_sec_offset
+ .byte 27 # DW_AT_comp_dir
+ .byte 14 # DW_FORM_strp
+ .byte 17 # DW_AT_low_pc
+ .byte 1 # DW_FORM_addr
+ .byte 18 # DW_AT_high_pc
+ .byte 6 # DW_FORM_data4
+ .byte 0 # EOM(1)
+ .byte 0 # EOM(2)
+ .byte 2 # Abbreviation Code
+ .byte 46 # DW_TAG_subprogram
+ .byte 0 # DW_CHILDREN_no
+ .byte 3 # DW_AT_name
+ .byte 14 # DW_FORM_strp
+ .byte 17 # DW_AT_low_pc
+ .byte 1 # DW_FORM_addr
+ .byte 18 # DW_AT_high_pc
+ .byte 6 # DW_FORM_data4
+ .byte 64 # DW_AT_frame_base
+ .byte 24 # DW_FORM_exprloc
+ .byte 71 # DW_AT_specification
+ .byte 16 # DW_FORM_ref_addr
+ .byte 0 # EOM(1)
+ .byte 0 # EOM(2)
+ .byte 3 # Abbreviation Code
+ .byte 36 # DW_TAG_base_type
+ .byte 0 # DW_CHILDREN_no
+ .byte 3 # DW_AT_name
+ .byte 14 # DW_FORM_strp
+ .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 4 # Abbreviation Code
+ .byte 46 # DW_TAG_subprogram
+ .byte 0 # DW_CHILDREN_no
+ .byte 58 # DW_AT_decl_file
+ .byte 11 # DW_FORM_data1
+ .byte 59 # DW_AT_decl_line
+ .byte 11 # DW_FORM_data1
+ .byte 39 # DW_AT_prototyped
+ .byte 25 # DW_FORM_flag_present
+ .byte 73 # DW_AT_type
+ .byte 19 # DW_FORM_ref4
+ .byte 63 # DW_AT_external
+ .byte 25 # DW_FORM_flag_present
+ .byte 60 # DW_AT_declaration
+ .byte 25 # DW_FORM_flag_present
+ .byte 0 # EOM(1)
+ .byte 0 # EOM(2)
+ .byte 0 # EOM(3)
+
+ .section .debug_info,"", at progbits
+.Ldebug_cu0:
+
+ .long .Ldebug_info_end0-.Ldebug_info_start0 # Length of Unit
+.Ldebug_info_start0:
+ .short 4 # DWARF version number
+ .long .debug_abbrev # Offset Into Abbrev. Section
+ .byte 8 # Address Size (in bytes)
+ .byte 1 # Abbrev [1] 0xb:0x40 DW_TAG_compile_unit
+ .long .Linfo_string_producer # DW_AT_producer
+ .short 12 # DW_AT_language
+ .long .Linfo_string_source # DW_AT_name
+ .long .Lline_table_start0 # DW_AT_stmt_list
+ .long .Linfo_string_directory # DW_AT_comp_dir
+ .quad .Lfunc_begin0 # DW_AT_low_pc
+ .long .Lfunc_end0-.Lfunc_begin0 # DW_AT_high_pc
+ .byte 2 # Abbrev [2] 0x2a:0x19 DW_TAG_subprogram
+ .long .Linfo_string_main # DW_AT_name
+ .quad .Lfunc_begin0 # DW_AT_low_pc
+ .long .Lfunc_end0-.Lfunc_begin0 # DW_AT_high_pc
+ .byte 1 # DW_AT_frame_base
+ .byte 86
+ .long .Ldebug_info1_main - .Ldebug_cu0 # DW_AT_specification
+ .byte 0 # End Of Children Mark
+.Ldebug_info_end0:
+
+.Ldebug_info1:
+ .long .Ldebug_info_end1-.Ldebug_info_start1 # Length of Unit
+.Ldebug_info_start1:
+ .short 4 # DWARF version number
+ .long .debug_abbrev # Offset Into Abbrev. Section
+ .byte 8 # Address Size (in bytes)
+ .byte 1 # Abbrev [1] 0xb:0x40 DW_TAG_compile_unit
+ .long .Linfo_string_producer # DW_AT_producer
+ .short 12 # DW_AT_language
+ .long .Linfo_string_source # DW_AT_name
+ .long .Lline_table_start1 # DW_AT_stmt_list
+ .long .Linfo_string_directory # DW_AT_comp_dir
+ .quad .Lfunc_begin0 # DW_AT_low_pc
+ .long .Lfunc_end0-.Lfunc_begin0 # DW_AT_high_pc
+.Ldebug_info1_main:
+ .byte 4 # Abbrev [2] 0x2a:0x19 DW_TAG_subprogram
+ .byte 1 # DW_AT_decl_file
+ .byte 1 # DW_AT_decl_line
+ # DW_AT_prototyped
+ .long .Ldebug_info1_int - .Ldebug_info1 # DW_AT_type
+ # DW_AT_external
+.Ldebug_info1_int:
+ .byte 3 # Abbrev [3] 0x43:0x7 DW_TAG_base_type
+ .long .Linfo_string_int # DW_AT_name
+ .byte 5 # DW_AT_encoding
+ .byte 4 # DW_AT_byte_size
+ .byte 0 # End Of Children Mark
+.Ldebug_info_end1:
+
+ .section .debug_str,"MS", at progbits,1
+.Linfo_string_producer:
+ .asciz "clang version 12.0.0" # string offset=0
+.Linfo_string_source:
+ .asciz "source.c" # string offset=130
+.Linfo_string_directory:
+ .asciz "/directory" # string offset=196
+.Linfo_string_main:
+ .asciz "main"
+.Linfo_string_int:
+ .asciz "int"
+ .ident "clang version 12.0.0"
+ .section ".note.GNU-stack","", at progbits
+ .addrsig
+
+ .section .debug_line,"", at progbits
+
+.Lline_table_start0:
+ .long .Lline_table_end0 - 1f # length from next byte
+1:
+ .short 4 # version
+ .long .Lline_table_end0 - 2f # prologue length from next byte
+2:
+ .byte 1 # minimum instruction length
+ .byte 1 # maximum ops per instruction
+ .byte 1 # initial value of is_stmt
+ .byte -5 # line base
+ .byte 14 # line range
+ .byte 1 # opcode base
+ # no standard opcodes
+ .asciz "directory0"
+ .byte 0 # last directory
+ .asciz "filename0"
+ .uleb128 1 # directory entry
+ .uleb128 0 # time
+ .uleb128 0 # file length
+ .byte 0 # last filename
+.Lline_table_end0:
+
+.Lline_table_start1:
+ .long .Lline_table_end1 - 1f # length from next byte
+1:
+ .short 4 # version
+ .long .Lline_table_end1 - 2f # prologue length from next byte
+2:
+ .byte 1 # minimum instruction length
+ .byte 1 # maximum ops per instruction
+ .byte 1 # initial value of is_stmt
+ .byte -5 # line base
+ .byte 14 # line range
+ .byte 1 # opcode base
+ # no standard opcodes
+ .asciz "directory1"
+ .byte 0 # last directory
+ .asciz "filename1"
+ .uleb128 1 # directory entry
+ .uleb128 0 # time
+ .uleb128 0 # file length
+ .byte 0 # last filename
+.Lline_table_end1:
More information about the lldb-commits
mailing list