From: Elena Zannoni <ezannoni@redhat.com>
To: Michael Snyder <msnyder@redhat.com>
Cc: gdb-patches@sources.redhat.com,
Elena Zannoni <ezannoni@redhat.com>,
Joern Rennecke <joern.rennecke@superh.com>
Subject: Re: [PATCH] sh64-tdep: regularize a few names
Date: Thu, 13 Nov 2003 01:50:00 -0000 [thread overview]
Message-ID: <16306.58205.167196.10248@localhost.redhat.com> (raw)
In-Reply-To: <3FB2C99E.2090809@redhat.com>
Michael Snyder writes:
> Just seems like "sh_sh64_register_name" is a trifle redundant.
> Makes them consistent with other method names in the same file.
>
Sigh, there was a scheme to all this apparent madness.
OK.
elena
> 2003-11-12 Michael Snyder <msnyder@redhat.com>
>
> * sh64_tdep.c: Regularize some names.
> (sh_sh64_register_name): Rename to sh64_register_name.
> (sh_sh64_breakpoint_from_pc): Rename to sh64_breakpoint_from_pc.
> (sh_sh64_register_byte): Rename to sh64_register_byte.
> (sh_sh64_build_float_register_type):
> Rename to sh64_build_float_register_type.
> (sh_sh64_register_convert_to_virtual):
> Rename to sh64_register_convert_to_virtual.
> (sh_sh64_register_convert_to_raw):
> Rename to sh64_register_convert_to_raw.
>
> Index: sh64-tdep.c
> ===================================================================
> RCS file: /cvs/src/src/gdb/sh64-tdep.c,v
> retrieving revision 1.14
> diff -p -w -b -r1.14 sh64-tdep.c
> *** sh64-tdep.c 12 Nov 2003 22:45:41 -0000 1.14
> --- sh64-tdep.c 12 Nov 2003 23:19:11 -0000
> *************** struct frame_extra_info
> *** 119,125 ****
> };
>
> static const char *
> ! sh_sh64_register_name (int reg_nr)
> {
> static char *register_names[] =
> {
> --- 119,125 ----
> };
>
> static const char *
> ! sh64_register_name (int reg_nr)
> {
> static char *register_names[] =
> {
> *************** pc_is_isa32 (bfd_vma memaddr)
> *** 248,254 ****
> }
>
> static const unsigned char *
> ! sh_sh64_breakpoint_from_pc (CORE_ADDR *pcptr, int *lenptr)
> {
> /* The BRK instruction for shmedia is
> 01101111 11110101 11111111 11110000
> --- 248,254 ----
> }
>
> static const unsigned char *
> ! sh64_breakpoint_from_pc (CORE_ADDR *pcptr, int *lenptr)
> {
> /* The BRK instruction for shmedia is
> 01101111 11110101 11111111 11110000
> *************** REGISTER_BYTE returns the register byte
> *** 1917,1923 ****
> */
> /* *INDENT-ON* */
> static int
> ! sh_sh64_register_byte (int reg_nr)
> {
> int base_regnum = -1;
>
> --- 1918,1924 ----
> */
> /* *INDENT-ON* */
> static int
> ! sh64_register_byte (int reg_nr)
> {
> int base_regnum = -1;
>
> *************** sh_sh64_register_byte (int reg_nr)
> *** 2002,2008 ****
> }
>
> static struct type *
> ! sh_sh64_build_float_register_type (int high)
> {
> struct type *temp;
>
> --- 2003,2009 ----
> }
>
> static struct type *
> ! sh64_build_float_register_type (int high)
> {
> struct type *temp;
>
> *************** sh64_register_type (struct gdbarch *gdba
> *** 2027,2038 ****
> return builtin_type_double;
> else if (reg_nr >= FPP0_REGNUM
> && reg_nr <= FPP_LAST_REGNUM)
> ! return sh_sh64_build_float_register_type (1);
> else if ((reg_nr >= FV0_REGNUM
> && reg_nr <= FV_LAST_REGNUM)
> ||(reg_nr >= FV0_C_REGNUM
> && reg_nr <= FV_LAST_C_REGNUM))
> ! return sh_sh64_build_float_register_type (3);
> else if (reg_nr == FPSCR_REGNUM)
> return builtin_type_int;
> else if (reg_nr >= R0_C_REGNUM
> --- 2028,2039 ----
> return builtin_type_double;
> else if (reg_nr >= FPP0_REGNUM
> && reg_nr <= FPP_LAST_REGNUM)
> ! return sh64_build_float_register_type (1);
> else if ((reg_nr >= FV0_REGNUM
> && reg_nr <= FV_LAST_REGNUM)
> ||(reg_nr >= FV0_C_REGNUM
> && reg_nr <= FV_LAST_C_REGNUM))
> ! return sh64_build_float_register_type (3);
> else if (reg_nr == FPSCR_REGNUM)
> return builtin_type_int;
> else if (reg_nr >= R0_C_REGNUM
> *************** sh64_register_type (struct gdbarch *gdba
> *** 2043,2049 ****
> }
>
> static void
> ! sh_sh64_register_convert_to_virtual (int regnum, struct type *type,
> char *from, char *to)
> {
> if (TARGET_BYTE_ORDER != BFD_ENDIAN_LITTLE)
> --- 2044,2050 ----
> }
>
> static void
> ! sh64_register_convert_to_virtual (int regnum, struct type *type,
> char *from, char *to)
> {
> if (TARGET_BYTE_ORDER != BFD_ENDIAN_LITTLE)
> *************** sh_sh64_register_convert_to_virtual (int
> *** 2068,2074 ****
> }
>
> static void
> ! sh_sh64_register_convert_to_raw (struct type *type, int regnum,
> const void *from, void *to)
> {
> if (TARGET_BYTE_ORDER != BFD_ENDIAN_LITTLE)
> --- 2069,2075 ----
> }
>
> static void
> ! sh64_register_convert_to_raw (struct type *type, int regnum,
> const void *from, void *to)
> {
> if (TARGET_BYTE_ORDER != BFD_ENDIAN_LITTLE)
> *************** sh64_pseudo_register_read (struct gdbarc
> *** 2114,2120 ****
> + register_size (gdbarch, base_regnum) * portion));
>
> /* We must pay attention to the endianness. */
> ! sh_sh64_register_convert_to_virtual (reg_nr,
> gdbarch_register_type (gdbarch,
> reg_nr),
> temp_buffer, buffer);
> --- 2115,2121 ----
> + register_size (gdbarch, base_regnum) * portion));
>
> /* We must pay attention to the endianness. */
> ! sh64_register_convert_to_virtual (reg_nr,
> gdbarch_register_type (gdbarch,
> reg_nr),
> temp_buffer, buffer);
> *************** sh64_pseudo_register_read (struct gdbarc
> *** 2186,2192 ****
> + register_size (gdbarch, base_regnum) * portion));
>
> /* We must pay attention to the endianness. */
> ! sh_sh64_register_convert_to_virtual (reg_nr,
> gdbarch_register_type (gdbarch,
> reg_nr),
> temp_buffer, buffer);
> --- 2187,2193 ----
> + register_size (gdbarch, base_regnum) * portion));
>
> /* We must pay attention to the endianness. */
> ! sh64_register_convert_to_virtual (reg_nr,
> gdbarch_register_type (gdbarch,
> reg_nr),
> temp_buffer, buffer);
> *************** sh64_pseudo_register_write (struct gdbar
> *** 2273,2280 ****
> {
> base_regnum = dr_reg_base_num (reg_nr);
> /* We must pay attention to the endianness. */
> ! sh_sh64_register_convert_to_raw (gdbarch_register_type (gdbarch,
> ! reg_nr), reg_nr,
> buffer, temp_buffer);
>
>
> --- 2274,2281 ----
> {
> base_regnum = dr_reg_base_num (reg_nr);
> /* We must pay attention to the endianness. */
> ! sh64_register_convert_to_raw (gdbarch_register_type (gdbarch, reg_nr),
> ! reg_nr,
> buffer, temp_buffer);
>
>
> *************** sh64_pseudo_register_write (struct gdbar
> *** 2348,2354 ****
> for (portion = 0; portion < 2; portion++)
> {
> /* We must pay attention to the endianness. */
> ! sh_sh64_register_convert_to_raw (gdbarch_register_type (gdbarch,
> reg_nr),
> reg_nr,
> buffer, temp_buffer);
> --- 2349,2355 ----
> for (portion = 0; portion < 2; portion++)
> {
> /* We must pay attention to the endianness. */
> ! sh64_register_convert_to_raw (gdbarch_register_type (gdbarch,
> reg_nr),
> reg_nr,
> buffer, temp_buffer);
> *************** sh64_gdbarch_init (struct gdbarch_info i
> *** 2870,2885 ****
> ((SIM_SH64_NR_FP_REGS + 1) * 4)
> + (SIM_SH64_NR_REGS - SIM_SH64_NR_FP_REGS -1) * 8);
>
> ! set_gdbarch_register_name (gdbarch, sh_sh64_register_name);
> set_gdbarch_register_type (gdbarch, sh64_register_type);
> set_gdbarch_deprecated_store_return_value (gdbarch, sh64_store_return_value);
> ! set_gdbarch_deprecated_register_byte (gdbarch, sh_sh64_register_byte);
> set_gdbarch_pseudo_register_read (gdbarch, sh64_pseudo_register_read);
> set_gdbarch_pseudo_register_write (gdbarch, sh64_pseudo_register_write);
>
> set_gdbarch_deprecated_do_registers_info (gdbarch, sh64_do_registers_info);
> set_gdbarch_deprecated_frame_init_saved_regs (gdbarch, sh64_nofp_frame_init_saved_regs);
> ! set_gdbarch_breakpoint_from_pc (gdbarch, sh_sh64_breakpoint_from_pc);
> set_gdbarch_deprecated_call_dummy_words (gdbarch, sh64_call_dummy_words);
> set_gdbarch_deprecated_sizeof_call_dummy_words (gdbarch, sizeof (sh64_call_dummy_words));
>
> --- 2871,2886 ----
> ((SIM_SH64_NR_FP_REGS + 1) * 4)
> + (SIM_SH64_NR_REGS - SIM_SH64_NR_FP_REGS -1) * 8);
>
> ! set_gdbarch_register_name (gdbarch, sh64_register_name);
> set_gdbarch_register_type (gdbarch, sh64_register_type);
> set_gdbarch_deprecated_store_return_value (gdbarch, sh64_store_return_value);
> ! set_gdbarch_deprecated_register_byte (gdbarch, sh64_register_byte);
> set_gdbarch_pseudo_register_read (gdbarch, sh64_pseudo_register_read);
> set_gdbarch_pseudo_register_write (gdbarch, sh64_pseudo_register_write);
>
> set_gdbarch_deprecated_do_registers_info (gdbarch, sh64_do_registers_info);
> set_gdbarch_deprecated_frame_init_saved_regs (gdbarch, sh64_nofp_frame_init_saved_regs);
> ! set_gdbarch_breakpoint_from_pc (gdbarch, sh64_breakpoint_from_pc);
> set_gdbarch_deprecated_call_dummy_words (gdbarch, sh64_call_dummy_words);
> set_gdbarch_deprecated_sizeof_call_dummy_words (gdbarch, sizeof (sh64_call_dummy_words));
>
prev parent reply other threads:[~2003-11-13 1:50 UTC|newest]
Thread overview: 2+ messages / expand[flat|nested] mbox.gz Atom feed top
2003-11-13 0:00 Michael Snyder
2003-11-13 1:50 ` Elena Zannoni [this message]
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=16306.58205.167196.10248@localhost.redhat.com \
--to=ezannoni@redhat.com \
--cc=gdb-patches@sources.redhat.com \
--cc=joern.rennecke@superh.com \
--cc=msnyder@redhat.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox