This is the mail archive of the binutils@sourceware.org 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]
Other format: [Raw text]

Re: [Patch, ARM, GAS] Fix apptance of APSR as an operand to MRC.


On Thu, 2010-09-16 at 10:45 +0100, Richard Earnshaw wrote:
> On Mon, 2010-09-13 at 14:08 +0100, Tejas Belagod wrote:
> > Hi,
> > 
> > Attached is a patch that fixes the acceptance of APSR_nzcv as a core
> > register operand to MRC instruction. This is the new syntax as specified
> > by UAL. In pre-UAL syntax, PC was written instead of APSR_nzcv to select
> > this form. This fix also has a small change to the disassembler where we
> > disassemble '1111' in the Rt field of MRC as 'APSR_nzcv' instead of
> > 'PC'. Tested for arm-none-eabi. OK?
> > 
> > --
> > Tejas Belagod
> > ARM.
> > 
> > gas/testsuite
> > 
> > 2010-09-13 Tejas Belagod <tejas.belagod@arm.com>
> > 
> > 	* gas/arm/copro.s: Add test for APSR_nzcv as an MRC operand.
> > 	* gas/arm/copro.d: Change pc in MRC to disassemble as
> > 	APSR_nzcv. Also add disassembly for test added in copro.s.
> > 
> > opcodes/
> > 
> > 2010-09-13 Tejas Belagod <tejas.belagod@arm.com>
> > 
> > 	* arm-dis.c (coprocessor_opcodes): Add MRC entry for APSR_nzcv.
> > 
> > gas/
> > 
> > 2010-09-13 Tejas Belagod <tejas.belagod@arm.com>
> > 
> > 	* config/tc-arm.c (insns): Change MRC entry to accept APSR_RR
> > 	instead of just RR.
> 
> This is OK
> 
> R.

I've now applied this on Tejas' behalf.  See:
http://sourceware.org/ml/binutils-cvs/2010-09/msg00104.html

Thanks,

Matt


-- 
Matthew Gretton-Dann
Principal Engineer - PDSW Tools
ARM Ltd


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