From d6fcebb67986070994b95fad6d715db6873c7fe7 Mon Sep 17 00:00:00 2001 From: palves Date: Tue, 25 Jan 2011 15:47:57 +0000 Subject: [PATCH] gdb/ * infcmd.c (default_print_registers_info): Allocate values so to never pass a NULL value to val_print. --- gdb/ChangeLog | 5 +++++ gdb/infcmd.c | 37 +++++++++++++++++++++++++------------ 2 files changed, 30 insertions(+), 12 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index be7b9c152d..0cf9411609 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,5 +1,10 @@ 2011-01-25 Pedro Alves + * infcmd.c (default_print_registers_info): Allocate values so to + never pass a NULL value to val_print. + +2011-01-25 Pedro Alves + * cp-valprint.c (cp_print_value): Treat the 'skip' local as boolean. Make sure to always pass a value that matches the contents buffer to callees. Preserve `address' for following diff --git a/gdb/infcmd.c b/gdb/infcmd.c index 500bdd7834..f589e3c10b 100644 --- a/gdb/infcmd.c +++ b/gdb/infcmd.c @@ -1942,10 +1942,12 @@ default_print_registers_info (struct gdbarch *gdbarch, int i; const int numregs = gdbarch_num_regs (gdbarch) + gdbarch_num_pseudo_regs (gdbarch); - gdb_byte buffer[MAX_REGISTER_SIZE]; for (i = 0; i < numregs; i++) { + struct type *regtype; + struct value *val; + /* Decide between printing all regs, non-float / vector regs, or specific reg. */ if (regnum == -1) @@ -1977,8 +1979,11 @@ default_print_registers_info (struct gdbarch *gdbarch, print_spaces_filtered (15 - strlen (gdbarch_register_name (gdbarch, i)), file); + regtype = register_type (gdbarch, i); + val = allocate_value (regtype); + /* Get the data in raw format. */ - if (! frame_register_read (frame, i, buffer)) + if (! frame_register_read (frame, i, value_contents_raw (val))) { fprintf_filtered (file, "*value not available*\n"); continue; @@ -1986,16 +1991,20 @@ default_print_registers_info (struct gdbarch *gdbarch, /* If virtual format is floating, print it that way, and in raw hex. */ - if (TYPE_CODE (register_type (gdbarch, i)) == TYPE_CODE_FLT - || TYPE_CODE (register_type (gdbarch, i)) == TYPE_CODE_DECFLOAT) + if (TYPE_CODE (regtype) == TYPE_CODE_FLT + || TYPE_CODE (regtype) == TYPE_CODE_DECFLOAT) { int j; struct value_print_options opts; + const gdb_byte *valaddr = value_contents_for_printing (val); get_user_print_options (&opts); opts.deref_ref = 1; - val_print (register_type (gdbarch, i), buffer, 0, 0, - file, 0, NULL, &opts, current_language); + + val_print (regtype, + value_contents_for_printing (val), + value_embedded_offset (val), 0, + file, 0, val, &opts, current_language); fprintf_filtered (file, "\t(raw 0x"); for (j = 0; j < register_size (gdbarch, i); j++) @@ -2006,7 +2015,7 @@ default_print_registers_info (struct gdbarch *gdbarch, idx = j; else idx = register_size (gdbarch, i) - 1 - j; - fprintf_filtered (file, "%02x", (unsigned char) buffer[idx]); + fprintf_filtered (file, "%02x", (unsigned char) valaddr[idx]); } fprintf_filtered (file, ")"); } @@ -2017,17 +2026,21 @@ default_print_registers_info (struct gdbarch *gdbarch, /* Print the register in hex. */ get_formatted_print_options (&opts, 'x'); opts.deref_ref = 1; - val_print (register_type (gdbarch, i), buffer, 0, 0, - file, 0, NULL, &opts, current_language); + val_print (regtype, + value_contents_for_printing (val), + value_embedded_offset (val), 0, + file, 0, val, &opts, current_language); /* If not a vector register, print it also according to its natural format. */ - if (TYPE_VECTOR (register_type (gdbarch, i)) == 0) + if (TYPE_VECTOR (regtype) == 0) { get_user_print_options (&opts); opts.deref_ref = 1; fprintf_filtered (file, "\t"); - val_print (register_type (gdbarch, i), buffer, 0, 0, - file, 0, NULL, &opts, current_language); + val_print (regtype, + value_contents_for_printing (val), + value_embedded_offset (val), 0, + file, 0, val, &opts, current_language); } } -- 2.11.0