0ffa8e959b
- Updated libstdc++ pretty printers to - Remove gdb-libstdc++-v3-python-6.3.1-20170212.tar.bz2 as unused in SUSE. - Add patches for s390x z14 feates [fate #321514, fate #322272] gdb-s390x-1b63490.patch gdb-s390x-289e23a.patch gdb-s390x-8fe09d7.patch gdb-s390x-96235dc.patch gdb-s390x-ad33963.patch - Adjust some patches: gdb-ppc-power7-test.patch gdb-rhbz795424-bitpos-20of25.patch gdb-rhbz795424-bitpos-21of25.patch gdb-vla-intel-fortran-vla-strings.patch - Add some patches from Fedora: gdb-rhbz1420304-s390x-01of35.patch gdb-rhbz1420304-s390x-02of35.patch gdb-rhbz1420304-s390x-03of35.patch gdb-rhbz1420304-s390x-04of35.patch gdb-rhbz1420304-s390x-05of35.patch gdb-rhbz1420304-s390x-06of35.patch gdb-rhbz1420304-s390x-07of35.patch gdb-rhbz1420304-s390x-08of35.patch gdb-rhbz1420304-s390x-09of35.patch gdb-rhbz1420304-s390x-10of35.patch gdb-rhbz1420304-s390x-11of35.patch gdb-rhbz1420304-s390x-12of35.patch gdb-rhbz1420304-s390x-13of35.patch gdb-rhbz1420304-s390x-14of35.patch OBS-URL: https://build.opensuse.org/package/show/devel:gcc/gdb?expand=0&rev=167
71 lines
2.4 KiB
Diff
71 lines
2.4 KiB
Diff
commit af547a9614969e1d1ea6fcec6b59cd77a606380f
|
|
Author: Andreas Arnez <arnez@linux.vnet.ibm.com>
|
|
Date: Tue Jun 13 15:20:27 2017 +0200
|
|
|
|
read/write_pieced_value: Respect value parent's offset
|
|
|
|
In the case of targeting a bit-field, read_pieced_value and
|
|
write_pieced_value calculate the number of bits preceding the bit-field
|
|
without considering the relative offset of the value's parent. This is
|
|
relevant for a structure variable like this:
|
|
|
|
struct s {
|
|
uint64_t foo;
|
|
struct {
|
|
uint32_t bar;
|
|
uint32_t bf : 10; /* <-- target bit-field */
|
|
} baz;
|
|
} s;
|
|
|
|
In this scenario, if 'val' is a GDB value representing s.baz.bf,
|
|
val->parent represents the whole s.baz structure, and the following holds:
|
|
|
|
- value_offset (val) == sizeof s.baz.bar == 4
|
|
- value_offset (val->parent) == sizeof s.foo == 8
|
|
|
|
The current logic would only use value_offset(val), resulting in the wrong
|
|
offset into the target value. This is fixed.
|
|
|
|
gdb/ChangeLog:
|
|
|
|
* dwarf2loc.c (read_pieced_value): Respect parent value's offset
|
|
when targeting a bit-field.
|
|
(write_pieced_value): Likewise.
|
|
|
|
### a/gdb/ChangeLog
|
|
### b/gdb/ChangeLog
|
|
## -1,5 +1,11 @@
|
|
2017-06-13 Andreas Arnez <arnez@linux.vnet.ibm.com>
|
|
|
|
+ * dwarf2loc.c (read_pieced_value): Respect parent value's offset
|
|
+ when targeting a bit-field.
|
|
+ (write_pieced_value): Likewise.
|
|
+
|
|
+2017-06-13 Andreas Arnez <arnez@linux.vnet.ibm.com>
|
|
+
|
|
* dwarf2loc.c (struct piece_closure) <addr_size>: Remove field.
|
|
(allocate_piece_closure): Drop addr_size parameter.
|
|
(dwarf2_evaluate_loc_desc_full): Adjust call to
|
|
--- a/gdb/dwarf2loc.c
|
|
+++ b/gdb/dwarf2loc.c
|
|
@@ -1776,7 +1776,8 @@ read_pieced_value (struct value *v)
|
|
bits_to_skip = 8 * value_offset (v);
|
|
if (value_bitsize (v))
|
|
{
|
|
- bits_to_skip += value_bitpos (v);
|
|
+ bits_to_skip += (8 * value_offset (value_parent (v))
|
|
+ + value_bitpos (v));
|
|
type_len = value_bitsize (v);
|
|
}
|
|
else
|
|
@@ -1946,7 +1947,8 @@ write_pieced_value (struct value *to, struct value *from)
|
|
bits_to_skip = 8 * value_offset (to);
|
|
if (value_bitsize (to))
|
|
{
|
|
- bits_to_skip += value_bitpos (to);
|
|
+ bits_to_skip += (8 * value_offset (value_parent (to))
|
|
+ + value_bitpos (to));
|
|
type_len = value_bitsize (to);
|
|
}
|
|
else
|