write_pieced_value: Transfer least significant bits into bit-field

On big-endian targets, when targeting a bit-field, write_pieced_value
currently transfers the source value's *most* significant bits to the
target value, instead of its least significant bits.  This is fixed.

In particular the fix adjusts the initial value of 'offset', which can now
potentially be nonzero.  Thus the variable 'type_len' is renamed to
'max_offset', to avoid confusion.  And for consistency, the affected logic
that was mirrored in read_pieced_value is changed there in the same way.

gdb/ChangeLog:

	* dwarf2loc.c (write_pieced_value): When writing to a bit-field,
	transfer the source value's least significant bits, instead of its
	lowest-addressed ones.  Rename type_len to max_offset.
	(read_pieced_value): Mirror above changes to write_pieced_value as
	applicable.
This commit is contained in:
Andreas Arnez 2017-06-13 15:20:28 +02:00
parent 07c9ca3bd8
commit 359b19bb24
2 changed files with 39 additions and 39 deletions

View File

@ -1,3 +1,11 @@
2017-06-13 Andreas Arnez <arnez@linux.vnet.ibm.com>
* dwarf2loc.c (write_pieced_value): When writing to a bit-field,
transfer the source value's least significant bits, instead of its
lowest-addressed ones. Rename type_len to max_offset.
(read_pieced_value): Mirror above changes to write_pieced_value as
applicable.
2017-06-13 Andreas Arnez <arnez@linux.vnet.ibm.com> 2017-06-13 Andreas Arnez <arnez@linux.vnet.ibm.com>
* dwarf2loc.c (write_pieced_value): In DWARF_VALUE_MEMORY, * dwarf2loc.c (write_pieced_value): In DWARF_VALUE_MEMORY,

View File

@ -1756,12 +1756,11 @@ static void
read_pieced_value (struct value *v) read_pieced_value (struct value *v)
{ {
int i; int i;
long offset = 0; LONGEST offset = 0, max_offset;
ULONGEST bits_to_skip; ULONGEST bits_to_skip;
gdb_byte *contents; gdb_byte *contents;
struct piece_closure *c struct piece_closure *c
= (struct piece_closure *) value_computed_closure (v); = (struct piece_closure *) value_computed_closure (v);
size_t type_len;
size_t buffer_size = 0; size_t buffer_size = 0;
std::vector<gdb_byte> buffer; std::vector<gdb_byte> buffer;
int bits_big_endian int bits_big_endian
@ -1778,12 +1777,12 @@ read_pieced_value (struct value *v)
{ {
bits_to_skip += (8 * value_offset (value_parent (v)) bits_to_skip += (8 * value_offset (value_parent (v))
+ value_bitpos (v)); + value_bitpos (v));
type_len = value_bitsize (v); max_offset = value_bitsize (v);
} }
else else
type_len = 8 * TYPE_LENGTH (value_type (v)); max_offset = 8 * TYPE_LENGTH (value_type (v));
for (i = 0; i < c->n_pieces && offset < type_len; i++) for (i = 0; i < c->n_pieces && offset < max_offset; i++)
{ {
struct dwarf_expr_piece *p = &c->pieces[i]; struct dwarf_expr_piece *p = &c->pieces[i];
size_t this_size, this_size_bits; size_t this_size, this_size_bits;
@ -1798,20 +1797,13 @@ read_pieced_value (struct value *v)
bits_to_skip -= this_size_bits; bits_to_skip -= this_size_bits;
continue; continue;
} }
if (bits_to_skip > 0) source_offset_bits = bits_to_skip;
{ this_size_bits -= bits_to_skip;
dest_offset_bits = 0; bits_to_skip = 0;
source_offset_bits = bits_to_skip; dest_offset_bits = offset;
this_size_bits -= bits_to_skip;
bits_to_skip = 0; if (this_size_bits > max_offset - offset)
} this_size_bits = max_offset - offset;
else
{
dest_offset_bits = offset;
source_offset_bits = 0;
}
if (this_size_bits > type_len - offset)
this_size_bits = type_len - offset;
this_size = (this_size_bits + source_offset_bits % 8 + 7) / 8; this_size = (this_size_bits + source_offset_bits % 8 + 7) / 8;
source_offset = source_offset_bits / 8; source_offset = source_offset_bits / 8;
@ -1932,12 +1924,11 @@ static void
write_pieced_value (struct value *to, struct value *from) write_pieced_value (struct value *to, struct value *from)
{ {
int i; int i;
long offset = 0;
ULONGEST bits_to_skip; ULONGEST bits_to_skip;
LONGEST offset = 0, max_offset;
const gdb_byte *contents; const gdb_byte *contents;
struct piece_closure *c struct piece_closure *c
= (struct piece_closure *) value_computed_closure (to); = (struct piece_closure *) value_computed_closure (to);
size_t type_len;
size_t buffer_size = 0; size_t buffer_size = 0;
std::vector<gdb_byte> buffer; std::vector<gdb_byte> buffer;
int bits_big_endian int bits_big_endian
@ -1949,12 +1940,20 @@ write_pieced_value (struct value *to, struct value *from)
{ {
bits_to_skip += (8 * value_offset (value_parent (to)) bits_to_skip += (8 * value_offset (value_parent (to))
+ value_bitpos (to)); + value_bitpos (to));
type_len = value_bitsize (to); /* Use the least significant bits of FROM. */
} if (gdbarch_byte_order (get_type_arch (value_type (from)))
== BFD_ENDIAN_BIG)
{
max_offset = 8 * TYPE_LENGTH (value_type (from));
offset = max_offset - value_bitsize (to);
}
else
max_offset = value_bitsize (to);
}
else else
type_len = 8 * TYPE_LENGTH (value_type (to)); max_offset = 8 * TYPE_LENGTH (value_type (to));
for (i = 0; i < c->n_pieces && offset < type_len; i++) for (i = 0; i < c->n_pieces && offset < max_offset; i++)
{ {
struct dwarf_expr_piece *p = &c->pieces[i]; struct dwarf_expr_piece *p = &c->pieces[i];
size_t this_size_bits, this_size; size_t this_size_bits, this_size;
@ -1968,20 +1967,13 @@ write_pieced_value (struct value *to, struct value *from)
bits_to_skip -= this_size_bits; bits_to_skip -= this_size_bits;
continue; continue;
} }
if (bits_to_skip > 0) dest_offset_bits = bits_to_skip;
{ this_size_bits -= bits_to_skip;
dest_offset_bits = bits_to_skip; bits_to_skip = 0;
source_offset_bits = 0; source_offset_bits = offset;
this_size_bits -= bits_to_skip;
bits_to_skip = 0; if (this_size_bits > max_offset - offset)
} this_size_bits = max_offset - offset;
else
{
dest_offset_bits = 0;
source_offset_bits = offset;
}
if (this_size_bits > type_len - offset)
this_size_bits = type_len - offset;
this_size = (this_size_bits + dest_offset_bits % 8 + 7) / 8; this_size = (this_size_bits + dest_offset_bits % 8 + 7) / 8;
source_offset = source_offset_bits / 8; source_offset = source_offset_bits / 8;