[patch] New m32r remote target, m32rsdi
Andrew Cagney
ac131313@redhat.com
Fri Sep 5 14:21:00 GMT 2003
> 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.
> I'm going to commit it next weak. Please let me know if there
> is any problem.
That wouldn't be a good idea.
Andrew
More information about the Gdb-patches
mailing list