diff --git a/gdb/ChangeLog b/gdb/ChangeLog index e5e6f2d4ce3..dd874209e77 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +2016-08-09 Pedro Alves + + * value.c (unpack_value_bitfield): Skip unpacking if the parent + has no contents buffer to begin with. + 2016-08-08 Pedro Alves * features/i386/amd64-avx-mpx-linux.c: Regenerate. diff --git a/gdb/testsuite/ChangeLog b/gdb/testsuite/ChangeLog index 9f336df109a..4b700591bf0 100644 --- a/gdb/testsuite/ChangeLog +++ b/gdb/testsuite/ChangeLog @@ -1,3 +1,7 @@ +2016-08-09 Pedro Alves + + * gdb.dwarf2/bitfield-parent-optimized-out.exp: New file. + 2016-08-03 Tom Tromey PR python/18565: diff --git a/gdb/testsuite/gdb.dwarf2/bitfield-parent-optimized-out.exp b/gdb/testsuite/gdb.dwarf2/bitfield-parent-optimized-out.exp new file mode 100644 index 00000000000..27d80448db6 --- /dev/null +++ b/gdb/testsuite/gdb.dwarf2/bitfield-parent-optimized-out.exp @@ -0,0 +1,87 @@ +# Copyright 2016 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 . + +# Test printing a bitfield of an optimized-out variable. Regression +# test for PR gdb/20295. + +load_lib dwarf.exp + +# This test can only be run on targets which support DWARF-2 and use gas. +if {![dwarf2_support]} { + return 0 +} + +standard_testfile main.c .S +set executable ${testfile} + +# Make some DWARF for the test. +set asm_file [standard_output_file $srcfile2] +Dwarf::assemble $asm_file { + global srcdir subdir srcfile + + cu { version 3 addr_size 4 } { + compile_unit {} { + declare_labels struct_label var_label int_label + + int_label: base_type { + {byte_size 4 sdata} + {encoding @DW_ATE_signed} + {name int} + } + + struct_label: structure_type { + {name S} + {byte_size 4 DW_FORM_sdata} + } { + member { + {name bitfield} + {type :$int_label} + {bit_size 12 DW_FORM_sdata} + {bit_offset 20 DW_FORM_sdata} + {data_member_location 2 DW_FORM_sdata} + } + member { + {name intfield} + {type :$int_label} + {data_member_location 2 DW_FORM_sdata} + } + } + + subprogram { + {MACRO_AT_func { main ${srcdir}/${subdir}/${srcfile} }} + {type :$int_label} + {external 1 flag} + } { + var_label: DW_TAG_variable { + {name var} + {location {} DW_FORM_block1} + {type :$struct_label} + } + } + } + } +} + +if [prepare_for_testing ${testfile}.exp $executable "${asm_file} ${srcfile}" {}] { + return -1 +} + +if ![runto_main] { + return -1 +} + +gdb_test "p var" " = " +gdb_test "p var.bitfield" " = " +gdb_test "p var.intfield" " = " diff --git a/gdb/value.c b/gdb/value.c index cd59f43f01f..d6eab24e62a 100644 --- a/gdb/value.c +++ b/gdb/value.c @@ -3430,17 +3430,24 @@ unpack_value_bitfield (struct value *dest_val, enum bfd_endian byte_order; int src_bit_offset; int dst_bit_offset; - LONGEST num; struct type *field_type = value_type (dest_val); - /* First, unpack and sign extend the bitfield as if it was wholly - available. Invalid/unavailable bits are read as zero, but that's - OK, as they'll end up marked below. */ byte_order = gdbarch_byte_order (get_type_arch (field_type)); - num = unpack_bits_as_long (field_type, valaddr + embedded_offset, - bitpos, bitsize); - store_signed_integer (value_contents_raw (dest_val), - TYPE_LENGTH (field_type), byte_order, num); + + /* First, unpack and sign extend the bitfield as if it was wholly + valid. Optimized out/unavailable bits are read as zero, but + that's OK, as they'll end up marked below. If the VAL is + wholly-invalid we may have skipped allocating its contents, + though. See allocate_optimized_out_value. */ + if (valaddr != NULL) + { + LONGEST num; + + num = unpack_bits_as_long (field_type, valaddr + embedded_offset, + bitpos, bitsize); + store_signed_integer (value_contents_raw (dest_val), + TYPE_LENGTH (field_type), byte_order, num); + } /* Now copy the optimized out / unavailability ranges to the right bits. */