From 912e805f749ef9ed7eda438d8b2349f7f5a06a78 Mon Sep 17 00:00:00 2001 From: Andrew Cagney Date: Sun, 5 Apr 1998 23:57:44 +0000 Subject: [PATCH] Declare array using MAX_REGISTER_RAW_SIZE, not REGISTER_RAW_SIZE(X) --- gdb/ChangeLog | 5 ++++ gdb/mips-tdep.c | 64 +++++++++++++++++++++++++++++++++---------------- 2 files changed, 49 insertions(+), 20 deletions(-) diff --git a/gdb/ChangeLog b/gdb/ChangeLog index 70f085deeaf..cbeb50c6adc 100644 --- a/gdb/ChangeLog +++ b/gdb/ChangeLog @@ -1,3 +1,8 @@ +Mon Apr 6 09:17:48 1998 Andrew Cagney + + * mips-tdep.c (mips_push_arguments): Specify dimention of valbuf + using MAX_REGISTER_RAW_SIZE. + Sat Apr 4 10:05:00 1998 Dawn Perchik * infrun.c: Fix prototype of signals_info to match static funtion. diff --git a/gdb/mips-tdep.c b/gdb/mips-tdep.c index 57f3bd76aa4..dc033a9a570 100644 --- a/gdb/mips-tdep.c +++ b/gdb/mips-tdep.c @@ -45,7 +45,7 @@ extern struct obstack frame_cache_obstack; static int mips_in_lenient_prologue PARAMS ((CORE_ADDR, CORE_ADDR)); #endif -static int gdb_print_insn_mips PARAMS ((bfd_vma, disassemble_info *)); +int gdb_print_insn_mips PARAMS ((bfd_vma, disassemble_info *)); static void mips_print_register PARAMS ((int, int)); @@ -1315,7 +1315,7 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr) for (argnum = 0; argnum < nargs; argnum++) { char *val; - char valbuf[REGISTER_RAW_SIZE(A0_REGNUM)]; + char valbuf[MAX_REGISTER_RAW_SIZE]; value_ptr arg = args[argnum]; struct type *arg_type = check_typedef (VALUE_TYPE (arg)); int len = TYPE_LENGTH (arg_type); @@ -1406,15 +1406,16 @@ mips_push_arguments(nargs, args, sp, struct_return, struct_addr) if (argreg > MIPS_LAST_ARG_REGNUM || odd_sized_struct) { /* Write this portion of the argument to the stack. */ - int longword_offset; + /* Should shorter than int integer values be + promoted to int before being stored? */ - longword_offset = 0; + int longword_offset = 0; if (TARGET_BYTE_ORDER == BIG_ENDIAN) if (MIPS_REGSIZE == 8 && (typecode == TYPE_CODE_INT || typecode == TYPE_CODE_PTR || typecode == TYPE_CODE_FLT) && len <= 4) - longword_offset = 4; + longword_offset = MIPS_REGSIZE - len; else if ((typecode == TYPE_CODE_STRUCT || typecode == TYPE_CODE_UNION) && TYPE_LENGTH (arg_type) < MIPS_REGSIZE) @@ -1757,14 +1758,23 @@ do_fp_register_row (regnum) static int do_gp_register_row (regnum) int regnum; -{ /* do values for GP (int) regs */ - char raw_buffer[REGISTER_RAW_SIZE(0)]; - int ncols = MIPS_REGSIZE == 8 ? 4 : 8; /* display cols per row */ - int col, byte, start_regnum = regnum; +{ + /* do values for GP (int) regs */ + char raw_buffer[MAX_REGISTER_RAW_SIZE]; + int ncols = (MIPS_REGSIZE == 8 ? 4 : 8); /* display cols per row */ + int col, byte; + int start_regnum = regnum; + int numregs = NUM_REGS; + +/* start-sanitize-sky */ +#ifdef NUM_R5900_REGS + numregs = NUM_R5900_REGS; +#endif +/* end-sanitize-sky */ /* For GP registers, we print a separate row of names above the vals */ printf_filtered (" "); - for (col = 0; col < ncols && regnum < NUM_REGS; regnum++) + for (col = 0; col < ncols && regnum < numregs; regnum++) { if (*reg_names[regnum] == '\0') continue; /* unused register */ @@ -1779,7 +1789,7 @@ do_gp_register_row (regnum) regnum = start_regnum; /* go back to start of row */ /* now print the values in hex, 4 or 8 to the row */ - for (col = 0; col < ncols && regnum < NUM_REGS; regnum++) + for (col = 0; col < ncols && regnum < numregs; regnum++) { if (*reg_names[regnum] == '\0') continue; /* unused register */ @@ -1788,6 +1798,9 @@ do_gp_register_row (regnum) /* OK: get the data in raw format. */ if (read_relative_register_raw_bytes (regnum, raw_buffer)) error ("can't read register %d (%s)", regnum, reg_names[regnum]); + /* pad small registers */ + for (byte = 0; byte < (MIPS_REGSIZE - REGISTER_RAW_SIZE (regnum)); byte++) + printf_filtered (" "); /* Now print the register value in hex, endian order. */ if (TARGET_BYTE_ORDER == BIG_ENDIAN) for (byte = 0; byte < REGISTER_RAW_SIZE (regnum); byte++) @@ -1823,13 +1836,23 @@ mips_do_registers_info (regnum, fpregs) { regnum = 0; while (regnum < NUM_REGS) - if (TYPE_CODE(REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT) - if (fpregs) /* true for "INFO ALL-REGISTERS" command */ - regnum = do_fp_register_row (regnum); /* FP regs */ + { + if (TYPE_CODE(REGISTER_VIRTUAL_TYPE (regnum)) == TYPE_CODE_FLT) + if (fpregs) /* true for "INFO ALL-REGISTERS" command */ + regnum = do_fp_register_row (regnum); /* FP regs */ + else + regnum += MIPS_NUMREGS; /* skip floating point regs */ else - regnum += MIPS_NUMREGS; /* skip floating point regs */ - else - regnum = do_gp_register_row (regnum); /* GP (int) regs */ + regnum = do_gp_register_row (regnum); /* GP (int) regs */ +/* start-sanitize-sky */ +#ifdef NUM_R5900_REGS + /* For the sky project, NUM_REGS includes the vector slaves, + which are handled elsewhere */ + if (regnum >= NUM_R5900_REGS) + break; +#endif +/* end-sanitize-sky */ + } } } @@ -2118,7 +2141,7 @@ mips_extract_return_value (valtype, regbuf, valbuf) regnum = 2; if (TYPE_CODE (valtype) == TYPE_CODE_FLT && (mips_fpu == MIPS_FPU_DOUBLE - || (mips_fpu == MIPS_FPU_SINGLE && len <= MIPS_REGSIZE))) + || (mips_fpu == MIPS_FPU_SINGLE && len <= MIPS_FPU_SINGLE_REGSIZE))) regnum = FP0_REGNUM; if (TARGET_BYTE_ORDER == BIG_ENDIAN) @@ -2338,7 +2361,7 @@ reinit_frame_cache_sfunc (args, from_tty, c) reinit_frame_cache (); } -static int +int gdb_print_insn_mips (memaddr, info) bfd_vma memaddr; disassemble_info *info; @@ -2644,7 +2667,8 @@ _initialize_mips_tdep () { struct cmd_list_element *c; - tm_print_insn = gdb_print_insn_mips; + if (!tm_print_insn) /* Someone may have already set it */ + tm_print_insn = gdb_print_insn_mips; /* Let the user turn off floating point and set the fence post for heuristic_proc_start. */ -- 2.30.2