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]

[RFA] Reenable mov lowreg, lowreg for Thumb when no -mcpu specified on command line.


All,

The recent change to disallow mov lowreg, lowreg in Thumb pre-v6 unfortunately stopped the assembler accepting this form of mov when no CPU had been specified on the command-line. This was noticed as the testsuite test "Invalid SP and PC operands test - THUMB" started failing.

The attached patch fixes this and also ensures we note we have used a v6 feature so that the output attributes are set correctly in this case.

This was tested arm-none-eabi.

Can someone please review?

Original patch which introduced the failure here:
  http://sourceware.org/ml/binutils/2011-11/msg00171.html

Thanks,

Matt

gas/ChangeLog:

2011-11-24 Matthew Gretton-Dann <matthew.gretton-dann@arm.com>

        * gas/config/tc-arm.c (do_t_mov_cmp): Allow MOV lowreg, lowreg
        when no CPU is specified.

gas/testsuite/ChangeLog:

2011-11-24 Matthew Gretton-Dann <matthew.gretton-dann@arm.com>

        * gas/arm/mov-highregs-any.d: New testcase.
        * gas/arm/mov-highregs-any.s: Likewise.
        * gas/arm/mov-lowregs-any.d: Likewise.
        * gas/arm/mov-lowregs-any.s: Likewise.

--
Matthew Gretton-Dann
Principal Engineer, PD Software - Tools, ARM Ltd

Attachment: 1111-lowregs.txt
Description: Text document


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