[PATCH] arm32: accept PUSH.W and POP.W

Jan Beulich JBeulich@suse.com
Mon Oct 7 09:55:00 GMT 2013


gas/
2013-10-07  Jan Beulich <jbeulich@suse.com>

	* gas/config/tc-arm.c (do_t_push_pop): Honor inst.size_req. Simplify
	LR/PC check.

gas/testsuite/
2013-10-07  Jan Beulich <jbeulich@suse.com>

	* gas/arm/thumb-w-good.s: Add PUSH.W and POP.W tests.
	* gas/arm/thumb-w-good.d: Update accordingly.

--- 2013-10-07/gas/config/tc-arm.c
+++ 2013-10-07/gas/config/tc-arm.c
@@ -11672,12 +11672,11 @@ do_t_push_pop (void)
 	      _("expression too complex"));
 
   mask = inst.operands[0].imm;
-  if ((mask & ~0xff) == 0)
+  if (inst.size_req != 4 && (mask & ~0xff) == 0)
     inst.instruction = THUMB_OP16 (inst.instruction) | mask;
-  else if ((inst.instruction == T_MNEM_push
-	    && (mask & ~0xff) == 1 << REG_LR)
-	   || (inst.instruction == T_MNEM_pop
-	       && (mask & ~0xff) == 1 << REG_PC))
+  else if (inst.size_req != 4
+	   && (mask & ~0xff) == (1 << (inst.instruction == T_MNEM_push
+				       ? REG_LR : REG_PC)))
     {
       inst.instruction = THUMB_OP16 (inst.instruction);
       inst.instruction |= THUMB_PP_PC_LR;
--- 2013-10-07/gas/testsuite/gas/arm/thumb-w-good.d
+++ 2013-10-07/gas/testsuite/gas/arm/thumb-w-good.d
@@ -7,3 +7,7 @@
 Disassembly of section .text:
 00000000 <.text> f7ff fffe 	bl	00000000 <foo>
 00000004 <.text\+0x4> f3ef 8000 	mrs	r0, CPSR
+00000008 <.text\+0x8> f84d 0d04 	(str(\.w)?	r0, \[sp, #-4\]!|push(\.w)?	\{r0\})
+0000000c <.text\+0xc> e92d 4001 	(stmdb(\.w)?	sp!,|push(\.w)?)[ 	]+\{r0, lr\}
+00000010 <.text\+0x10> f85d 0b04 	(ldr(\.w)?	r0, \[sp\], #4|pop(\.w)?	\{r0\})
+00000014 <.text\+0x14> e8bd 8001 	(ldmia(\.w)?	sp!,|pop(\.w)?)[ 	]+\{r0, pc\}
--- 2013-10-07/gas/testsuite/gas/arm/thumb-w-good.s
+++ 2013-10-07/gas/testsuite/gas/arm/thumb-w-good.s
@@ -5,3 +5,8 @@ bl.w foo
 .arch armv6-m
 mrs.w r0, apsr
 
+.arch armv6t2
+push.w {r0}
+push.w {r0, lr}
+pop.w {r0}
+pop.w {r0, pc}



-------------- next part --------------
An embedded and charset-unspecified text was scrubbed...
Name: binutils-mainline-arm-push-pop-wide.patch
URL: <https://sourceware.org/pipermail/binutils/attachments/20131007/cc382887/attachment.ksh>


More information about the Binutils mailing list