This is the mail archive of the gdb-patches@sources.redhat.com mailing list for the GDB project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [patch] New m32r remote target, m32rsdi


From: "Andrew Cagney" <ac131313@redhat.com>
To: "Kei Sakamoto" <sakamoto.kei@renesas.com>
Cc: <gdb-patches@sources.redhat.com>
Sent: Friday, September 05, 2003 11:21 PM
Subject: Re: [patch] New m32r remote target, m32rsdi


> > Hello,
> > 
> > I posted the following two weeks ago, but there is no reaction
> > so far.
> > 
> > Its patch adds a new remote protocol to m32r architecture and
> > does not influence any other functions. So I think it's safe
> > to add.
> 
> Dig dig, ah I see Daniel wrote:
> 
> > I don't feel qualified to review a new remote target, so you'll have to
> > ask the remote maintainer.  However, there's definitely a problem with
> > the new code.  It's full of pieces like:
> > 
> >   if (TARGET_BYTE_ORDER == BFD_ENDIAN_BIG)
> >     *((unsigned long *) (buf + 1)) = pc_addr;
> >   else
> >     *((unsigned long *) (buf + 1)) = pc_addr - 1;
> > 
> > This sends host-endian data over the wire to the target.  Run it on a
> > big-endian host and bad things will happen.
> 
> I guess that means me (but I'm only listed for remote.c :-).  I'll look
> over the revision.

Thank you for reviewing my patch!

> > I'm going to commit it next weak. Please let me know if there
> > is any problem.
> 
> That wouldn't be a good idea.

I see. I'll commit it after your approval.

Kei Sakamoto


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]