This is the mail archive of the
gdb-patches@sourceware.org
mailing list for the GDB project.
Re: RFA: contribute Renesas M32C sim
- From: Daniel Jacobowitz <drow at false dot org>
- To: gdb-patches at sourceware dot org
- Date: Tue, 13 Dec 2005 17:47:22 -0500
- Subject: Re: RFA: contribute Renesas M32C sim
- References: <m3zmpl5a7a.fsf@alligator.red-bean.com> <20051009201500.GD7107@nevyn.them.org> <vt2fyr6sij6.fsf@theseus.home.> <200512132243.jBDMh6MN025311@greed.delorie.com>
On Tue, Dec 13, 2005 at 05:43:06PM -0500, DJ Delorie wrote:
>
> > >> + void
> > >> + prefix (src_allowed, dest_allowed, index_bytewidth)
> > >> + {
> > >> + }
> > >
> > > And what's that for?
> >
> > I don't know. I thought it was some sort of annotation for opc2c to
> > consume, but that doesn't seem to be so. I'll find out.
>
> I remember talking to Jim about this, but I don't see him answering in
> the mail archives, and since I was checking on the m32c/gdb status I
> figured I'd at least get this one answered.
>
> In short, that function exists so it can be called, it's the callers
> that are important at the moment.
>
> The callers act as documentation for what prefixes and indexes are
> permitted for each opcode. As I was going through the tedious process
> of implementing each opcode, I added all the calls to prefix() with
> the correct arguments. The function does nothing yet, but at some
> point in the future it may fault if prefixes are used where the real
> chip doesn't support them.
Then please add some useful information there, so that the next
reviewer through the code does not remove it :-)
--
Daniel Jacobowitz
CodeSourcery, LLC