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: Re: [PATCH, ARM] MSR/MRS assembly and disassembly tweaks

From: Paul Carroll <pcarroll at codesourcery dot com>
To: Tejas Belagod <tejas dot belagod at arm dot com>
Cc: Julian Brown <julian at codesourcery dot com>, binutils at sourceware dot org
Date: Tue, 12 Apr 2011 22:33:10 -0600
Subject: Re: Re: [PATCH, ARM] MSR/MRS assembly and disassembly tweaks
References: <1302626484.21908.5.camel@e102484-lin.cambridge.arm.com>

On 12:59 PM, Tejas Belagod wrote:

Hi Julian,

This patch seems to break one of the existing tests - msr-imm.s.
Basically it now thinks these instructions are illegal.

$ as-new -march=mv7-a t.s
t.s: Assembler messages:
t.s:5: Error: syntax error -- `msr APSR_nzcvqg,#0x00000004'
t.s:6: Error: syntax error -- `msr APSR_g,#0x00000000'
t.s:7: Error: syntax error -- `msr APSR_nzcvq,#0x00000004'

Actually, I believe the 4 new gas test failures in the ARM toolkit arise from changes for bug patch 12296, where include/opcode/arm.h was modified to add ARM_EXT_OS to the ARM_AEXT_V7_ARM definition. I saw this issue when comparing two source pulls, one from Saturday and one from Monday.

Follow-Ups:
- Re: [PATCH, ARM] MSR/MRS assembly and disassembly tweaks
  - From: Julian Brown

References:
- Re: [PATCH, ARM] MSR/MRS assembly and disassembly tweaks
  - From: Tejas Belagod

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