Fix some MIPS operand typos

Maciej W. Rozycki macro@codesourcery.com
Wed Jun 26 10:09:00 GMT 2013


On Wed, 26 Jun 2013, Richard Sandiford wrote:

> > binutils-mips-include-ops-doc.diff
> > Index: binutils-fsf-trunk-quilt/include/opcode/mips.h
> > ===================================================================
> > --- binutils-fsf-trunk-quilt.orig/include/opcode/mips.h	2013-06-25 01:09:04.000000000 +0100
> > +++ binutils-fsf-trunk-quilt/include/opcode/mips.h	2013-06-25 23:50:55.211225703 +0100
> > @@ -441,14 +441,14 @@ struct mips_opcode
> >     "W" 5 bit same register used as floating target and destination (OP_*_FT)
> >  
> >     Coprocessor instructions:
> > -   "E" 5 bit target register (OP_*_RT)
> > -   "G" 5 bit destination register (OP_*_RD)
> > +   "E" 5 bit general register (OP_*_RT)
> > +   "G" 5 bit coprocessor register (OP_*_RD)
> 
> "E" is a coprocessor register, not a general register.

 Indeed.

> The current "E" seems fine to me.  It's consistent with "t", etc.,
> and we have RD_t for a reason.

 That refers to the field name, that stays the same across the instruction 
set, and not its function with a particular set of opcodes though.

> In case this response tempts you to try a wholesale edit of the format
> comments: please hold off for now.  The reason I noticed in the first
> place is that I'm working on a series to change the way we handle this
> stuff, and that series might well und up changing these comments anyway.
> I just wanted to fix wrong info as I found it.

 OK.

> I applied the patch below, which includes another mismatch I found later.

 Thanks.

  Maciej



More information about the Binutils mailing list