Remove MAX_REGISTER_SIZE from frame.c
Commit Message
In addition, remove as usage of deprecated_frame_register_read
Tested using make check with board files unix and native-gdbserver.
Ok to commit?
Alan.
2017-02-21 Alan Hayward <alan.hayward@arm.com>
* gdb/frame.c (frame_unwind_register_signed): Use LONGEST for max size.
(frame_unwind_register_unsigned): Use ULONGEST for max size.
(get_frame_register_bytes): Unwind using value.
(put_frame_register_bytes): Likewise.
@@ -1252,7 +1252,11 @@ frame_unwind_register_signed (struct frame_info *frame, int regnum)
struct gdbarch *gdbarch = frame_unwind_arch (frame);
enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
int size = register_size (gdbarch, regnum);
- gdb_byte buf[MAX_REGISTER_SIZE];
+ gdb_byte buf[sizeof (LONGEST)];
+
+ if (size > (int) sizeof (LONGEST))
+ error (_("Cannot unwind integers more than %d bytes."),
+ (int) sizeof (LONGEST));
frame_unwind_register (frame, regnum, buf);
return extract_signed_integer (buf, size, byte_order);
@@ -1270,7 +1274,11 @@ frame_unwind_register_unsigned (struct frame_info *frame, int regnum)
struct gdbarch *gdbarch = frame_unwind_arch (frame);
enum bfd_endian byte_order = gdbarch_byte_order (gdbarch);
int size = register_size (gdbarch, regnum);
- gdb_byte buf[MAX_REGISTER_SIZE];
+ gdb_byte buf[sizeof (ULONGEST)];
+
+ if (size > (int) sizeof (ULONGEST))
+ error (_("Cannot unwind integers more than %d bytes."),
+ (int) sizeof (ULONGEST));
frame_unwind_register (frame, regnum, buf);
return extract_unsigned_integer (buf, size, byte_order);
@@ -1410,16 +1418,21 @@ get_frame_register_bytes (struct frame_info *frame, int regnum,
}
else
{
- gdb_byte buf[MAX_REGISTER_SIZE];
- enum lval_type lval;
- CORE_ADDR addr;
- int realnum;
+ struct value *value = frame_unwind_register_value (frame->next,
+ regnum);
+ gdb_assert (value != NULL);
+ *optimizedp = value_optimized_out (value);
+ *unavailablep = !value_entirely_available (value);
- frame_register (frame, regnum, optimizedp, unavailablep,
- &lval, &addr, &realnum, buf);
if (*optimizedp || *unavailablep)
- return 0;
- memcpy (myaddr, buf + offset, curr_len);
+ {
+ release_value (value);
+ value_free (value);
+ return 0;
+ }
+ memcpy (myaddr, value_contents_all (value) + offset, curr_len);
+ release_value (value);
+ value_free (value);
}
myaddr += curr_len;
@@ -1460,11 +1473,15 @@ put_frame_register_bytes (struct frame_info *frame, int regnum,
}
else
{
- gdb_byte buf[MAX_REGISTER_SIZE];
-
- deprecated_frame_register_read (frame, regnum, buf);
- memcpy (buf + offset, myaddr, curr_len);
- put_frame_register (frame, regnum, buf);
+ struct value *value = frame_unwind_register_value (frame->next,
+ regnum);
+ gdb_assert (value != NULL);
+
+ memcpy ((char *) value_contents_all (value) + offset, myaddr,
+ curr_len);
+ put_frame_register (frame, regnum, value_contents_all (value));
+ release_value (value);
+ value_free (value);
}
myaddr += curr_len;