This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC 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/Thumb1]Fix 'mov' instruction for Thumb-1 UAL

From: Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>
To: Terry Guo <terry dot guo at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
Date: Tue, 11 Nov 2014 08:41:09 +0000
Subject: Re: [Patch,ARM/Thumb1]Fix 'mov' instruction for Thumb-1 UAL
Authentication-results: sourceware.org; auth=none
References: <000101cffd8b$1fcbb8c0$5f632a40$ at arm dot com>



On 11/11/14 08:40, Terry Guo wrote:

Hi there,

Attached patch intends to fix below trunk failure caused by recent thumb-1
UAL patch:

/tmp/cc9EfnXy.s: Assembler messages:
/tmp/cc9EfnXy.s:69: Error: MOV Rd, Rs with two low registers is not
permitted on this architecture -- `mov r6,r7'

Now for pre-v6 Thumb-1, the 'movs' will be used rather than the 'mov'.

The multilib for ARM/Thumb1/hard-float all can be built. Tested with
regression test on armv4t thumb and v6m thumb. No regression. Is it ok to
trunk?


This is OK.

Ramana


BR,
Terry

2014-11-11  Terry Guo  <terry.guo@arm.com>

      * doc/invoke.texi (-masm-syntax-unified): Reword and fix typo.
      * config/arm/thumb1.md (*thumb_mulsi3): Use movs to move low registers.
      (*thumb1_movhf): Likewise.

References:
- [Patch,ARM/Thumb1]Fix 'mov' instruction for Thumb-1 UAL
  - From: Terry Guo

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