From: Andrew Cagney <ac131313@cygnus.com>
To: joern.rennecke@st.com
Cc: bje@redhat.com, gdb-patches@sources.redhat.com
Subject: Re: SH5 simulator contribution
Date: Thu, 18 Apr 2002 18:32:00 -0000 [thread overview]
Message-ID: <3CBF73A3.2090409@cygnus.com> (raw)
In-Reply-To: <3CBA940B.B99F0E4C@st.com>
> So I don't see that you gain anything by unifying the numbering scheme
>> > in the gdb <-> sim interface, as it would be at odds with the interface
>> > to gcc and the hardware interfaces.
>
>>
>> Formalizing would be a better word. So that GDB and the SIM can agree
>> on the register numbers and their sizes without needing to know the
>> others internals.
>
>
> They only need to know if the program is for an sh5 or an earlier processor.
> This information is readily available from the elf flags (the lower five bits
> of which enconde the sh version this program is compiled for), or the bfd
> word size. Gdb already needs to know this in order to print registers in the
> correct size, and the simulator in order to simulate the right instruction
> set(s).
Yes?
> So do you want a file that explicitly documents the two interface?
> The danger of this is that if registers are added to a successor of the sh5,
> the documentation can get out-of-sync with the header file.
Sorry, you've lost me here. There should be only one GDB:SIM interface
for the SH.
> Or should we rather make an include/sim-sh.h file - to be used in the old
> simulators sim_fetch_register / sim_store_register as well as in the
> to-be-written translation function in the new simulator for sh2-sh4 programs,
> and state in gdbint.texi that the interfaces are defined in include/sim-sh.h
> and include/sim-sh64.h ?
Having just looked at a different target (similar problem), I think
having a single file that assigns different number ranges to the sh4 vs
sh64 registers would be best. That would make it easy to detect things
like trying to fetch an SH64 register from the SH4 sim.
Andrew
next prev parent reply other threads:[~2002-04-19 1:32 UTC|newest]
Thread overview: 21+ messages / expand[flat|nested] mbox.gz Atom feed top
2002-02-02 2:06 Ben Elliston
2002-02-04 19:48 ` Andrew Cagney
2002-02-04 20:28 ` Ben Elliston
2002-02-04 20:59 ` Andrew Cagney
2002-02-04 22:29 ` Ben Elliston
2002-02-05 8:31 ` Andrew Cagney
2002-02-05 12:21 ` Ben Elliston
2002-02-05 17:36 ` Andrew Cagney
2002-04-12 2:46 ` Joern Rennecke
2002-04-12 9:30 ` Andrew Cagney
2002-04-12 9:45 ` Andrew Cagney
2002-04-15 1:48 ` Joern Rennecke
2002-04-18 18:32 ` Andrew Cagney [this message]
2002-04-29 10:23 ` Joern Rennecke
2002-04-29 10:47 ` Andrew Cagney
2002-04-29 11:30 ` Joern Rennecke
2002-04-12 2:48 ` Joern Rennecke
2002-04-12 2:57 ` Joern Rennecke
2002-04-12 2:57 ` Joern Rennecke
2002-04-12 2:57 ` Joern Rennecke
2002-04-12 2:58 ` Joern Rennecke
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=3CBF73A3.2090409@cygnus.com \
--to=ac131313@cygnus.com \
--cc=bje@redhat.com \
--cc=gdb-patches@sources.redhat.com \
--cc=joern.rennecke@st.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