[Patch, microblaze, gas, opcodes] Change insn type INST_TYPE_RD_R1_SPECIAL -> INST_TYPE_R1_R2_SPECIAL
David Holsgrove
david.holsgrove@xilinx.com
Thu Nov 29 06:17:00 GMT 2012
Hi Michael,
> -----Original Message-----
> From: Michael Eager [mailto:eager@eagerm.com]
> Sent: Thursday, 29 November 2012 5:15 am
> To: David Holsgrove
> Cc: binutils@sourceware.org; John Williams; Vinod Kathail; Tom Shui; Vidhumouli
> Hunsigida; Nagaraju Mekala; Edgar E. Iglesias
> Subject: Re: [Patch, microblaze, gas, opcodes] Change insn type
> INST_TYPE_RD_R1_SPECIAL -> INST_TYPE_R1_R2_SPECIAL
>
> On 11/22/2012 04:04 AM, David Holsgrove wrote:
> >
> > Change write to data cache and write to instruction cache insns to be
> > of type INST_TYPE_R1_R2_SPECIAL (currently INST_TYPE_RD_R1_SPECIAL)
>
> Is this patch only renaming INST_TYPE_RD_R1_SPECIAL to
> INST_TYPE_R1_R2_SPECIAL
> other than the following change?
>
> /* insn wic ra, rb => wic ra, ra, rb. */
> - inst |= (reg1 << RD_LOW) & RD_MASK;
> inst |= (reg1 << RA_LOW) & RA_MASK;
> inst |= (reg2 << RB_LOW) & RB_MASK;
>
> I don't see changes to wdc* instructions.
>
Thanks for the review - yes this patch is renaming INST_TYPE_RD_R1_SPECIAL to INST_TYPE_R1_R2_SPECIAL,
and then in opcodes/microblaze-opc.h we switch the instructions wic, wdc* to use this newly defined type.
thanks,
David
> --
> Michael Eager eager@eagercon.com
> 1960 Park Blvd., Palo Alto, CA 94306 650-325-8077
>
>
More information about the Binutils
mailing list