diff options
author | Tom de Vries <tdevries@suse.de> | 2023-09-20 16:05:55 +0200 |
---|---|---|
committer | Tom de Vries <tdevries@suse.de> | 2023-09-20 16:05:55 +0200 |
commit | 973db6fae304c2bcafff5f169610f663adc9e817 (patch) | |
tree | 02062c89d78e1332ab3a57cc826c77a357f4a7a1 /gdb/testsuite | |
parent | fbf4b3c9bb0347950ee8792093e44c9d0cac60c5 (diff) | |
download | gdb-973db6fae304c2bcafff5f169610f663adc9e817.zip gdb-973db6fae304c2bcafff5f169610f663adc9e817.tar.gz gdb-973db6fae304c2bcafff5f169610f663adc9e817.tar.bz2 |
[gdb/symtab] Error out for .debug_types section in dwz file
There are two methods to factor out type information in a dwarf4 executable:
- use -fdebug-info-types to generate type units in a .debug_types section, and
- use dwz to create partial units.
The dwz method has an extra benefit: it also allows to factor out information
between executables into a newly created .dwz file, pointed to by a
.gnu_debugaltlink section.
There is nothing prohibiting a .gnu_debugaltlink file to contain a
.debug_types section.
It's just not generated by dwz or any other tool atm, and consequently gdb has
no support for it. Enhancement PR symtab/30838 is open about the lack of
support.
Make the current situation explicit by emitting a dwarf error:
...
(gdb) file struct-with-sig-2^M
Reading symbols from struct-with-sig-2...^M
Dwarf Error: .debug_types section not supported in dwz file^M
...
and add an assert in write_gdbindex:
...
+ /* See enhancement PR symtab/30838. */
+ gdb_assert (!(per_cu->is_dwz && per_cu->is_debug_types));
...
to clarify why we can use:
...
data_buf &cu_list = (per_cu->is_debug_types
? types_cu_list
: per_cu->is_dwz ? dwz_cu_list : objfile_cu_list);
...
The test-case is a modified copy from gdb.dwarf2/struct-with-sig.exp, so it
keeps the copyright years range.
Tested on x86_64-linux.
Tested-By: Guinevere Larsen <blarsen@redhat.com>
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=30838
Diffstat (limited to 'gdb/testsuite')
-rw-r--r-- | gdb/testsuite/gdb.dwarf2/struct-with-sig-2.exp | 128 |
1 files changed, 128 insertions, 0 deletions
diff --git a/gdb/testsuite/gdb.dwarf2/struct-with-sig-2.exp b/gdb/testsuite/gdb.dwarf2/struct-with-sig-2.exp new file mode 100644 index 0000000..f6877fc --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/struct-with-sig-2.exp @@ -0,0 +1,128 @@ +# Copyright 2020-2023 Free Software Foundation, Inc. + +# This program is free software; you can redistribute it and/or modify +# it under the terms of the GNU General Public License as published by +# the Free Software Foundation; either version 3 of the License, or +# (at your option) any later version. +# +# This program is distributed in the hope that it will be useful, +# but WITHOUT ANY WARRANTY; without even the implied warranty of +# MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the +# GNU General Public License for more details. +# +# You should have received a copy of the GNU General Public License +# along with this program. If not, see <http://www.gnu.org/licenses/>. + +# Variant of struct-with-sig.exp where TUs are in .gnu_debugaltlink file. + +load_lib dwarf.exp + +# This test can only be run on targets which support DWARF-2 and use gas. +require dwarf2_support + +standard_testfile main-foo.c .S .S + +set build_id [string repeat ff 20] + +# Make some DWARF for the dwz file. +set asm_file [standard_output_file $srcfile2] +Dwarf::assemble $asm_file { + build_id $::build_id + + tu {} 0x0000000000000001 the_type_i { + type_unit {} { + declare_labels int_type + + the_type_i: structure_type { + {name s} + {byte_size 4 sdata} + } { + member { + {name i} + {type :$int_type} + } + } + int_type: base_type { + {name int} + {encoding @DW_ATE_signed} + {byte_size 4 sdata} + } + } + } + + tu {} 0x0000000000000002 the_type_j { + type_unit {} { + declare_labels int_type + + the_type_j: structure_type { + {name s} + {byte_size 4 sdata} + } { + member { + {name j} + {type :$int_type} + } + } + int_type: base_type { + {name int} + {encoding @DW_ATE_signed} + {byte_size 4 sdata} + } + } + } +} + +# We're using an object instead of executable here to keep cc-with-tweaks.sh +# from modifying it. It also means we can set the build-id, rather than +# having to extract it. +if { [gdb_compile $asm_file $binfile.dwz object {nodebug}] != "" } { + return -1 +} + +set host_dwz_file [gdb_remote_download host $binfile.dwz] + +# Make some DWARF for the executable. +set asm_file [standard_output_file $srcfile3] +Dwarf::assemble $asm_file { + gnu_debugaltlink $::host_dwz_file $::build_id + + cu {} { + compile_unit { + {DW_AT_language @DW_LANG_C} + {DW_AT_name main.c} + } { + structure_type { + {name s} + {signature 0x0000000000000001 ref_sig8} + {declaration 1 flag} + } + DW_TAG_subprogram { + {MACRO_AT_func {main}} + } + } + } + + cu {} { + compile_unit { + {DW_AT_language @DW_LANG_C} + {DW_AT_name foo.c} + } { + structure_type { + {name s} + {signature 0x0000000000000002 ref_sig8} + {declaration 1 flag} + } + DW_TAG_subprogram { + {MACRO_AT_func {foo}} + } + } + } +} + +if { [prepare_for_testing "failed to prepare" ${testfile} \ + [list $srcfile $asm_file] {nodebug}] } { + return -1 +} + +set re "Dwarf Error: .debug_types section not supported in dwz file" +gdb_assert { [regexp $re $gdb_file_cmd_msg] } "Dwarf Error message" |