This is the mail archive of the binutils@sources.redhat.com mailing list for the binutils project.


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

Re: ppc breakage


On Fri, Oct 19, 2001 at 02:10:57AM +1000, matthew green wrote:
> 
> yah, i need a BOOKE entry for sure -- that's what i meant above
> by "thought i had fixed this."  but the current one needs to
> move out of "PPC" otherwise you get, eg:
> 
> 	Error: Internal assembler error for instruction dcbtst
> 
> 
> i'm just not sure what CPU's actually support that version of
> dcbt and dcbtst...

All ppc cpus will accept the instruction, but it may be a nop.  After
talking this over with Paul Mackerras, it seems like the right thing to
do is:

opcodes/ChangeLog
	* ppc-opc.c (CT): Make it an optional operand.

-- 
Alan Modra

Index: opcodes/ppc-opc.c
===================================================================
RCS file: /cvs/src/src/opcodes/ppc-opc.c,v
retrieving revision 1.21
diff -u -p -r1.21 ppc-opc.c
--- ppc-opc.c	2001/10/17 13:13:16	1.21
+++ ppc-opc.c	2001/10/19 00:45:31
@@ -195,7 +195,7 @@ const struct powerpc_operand powerpc_ope
 
   /* The CT field in an X form instruction.  */
 #define CT CR + 1
-  { 5, 21, 0, 0, 0 },
+  { 5, 21, 0, 0, PPC_OPERAND_OPTIONAL },
 
   /* The D field in a D form instruction.  This is a displacement off
      a register, and implies that the next operand is a register in


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