[GAS] [PATCH] Added support for the ARM VMSR and VMRS instructions

Nick Clifton nickc@redhat.com
Thu Nov 12 10:03:00 GMT 2009


Hi Viktor,

> Attached is a patch that adds a support for the ARM VMSR and VMRS 
> instructions.

The patch is mostly OK, but there are a few issues:

   --- gas/config/tc-arm.c	3 Nov 2009 12:37:44 -0000	1.413
   +++ gas/config/tc-arm.c	11 Nov 2009 02:22:49 -0000
   @@ -6303,6 +6395,7 @@
                  if (found != 15)
                    goto failure;
                  inst.operands[i].isvec = 1;
   +	      inst.operands[i].reg = 15;
                }
              else
                goto failure;

What is the purpose of this change ?


   +  /* APSR_ sets isvec. All other refs to PC are illegal */

Comment formatting.  Should end:  illegal.  */


One other point.  Please could you add a couple of lines to 
gas/testsuite/gas/arm/vfp1xD.s and vfp1xD.d to check the assembly of the 
vmsr and vmrs instructions ?

Cheers
   Nick



More information about the Binutils mailing list