RE: [PATCH][GAS][ARM] Autodetection tests

["Thomas Preud'homme" <thomas dot preudhomme at foss dot arm dot com>]

Committed as well.

Best regards,

Thomas

> -----Original Message-----
> From: binutils-owner@sourceware.org [mailto:binutils-
> owner@sourceware.org] On Behalf Of Richard Earnshaw (lists)
> Sent: Wednesday, December 23, 2015 11:10 PM
> To: Thomas Preud'homme; binutils@sourceware.org
> Subject: Re: [PATCH][GAS][ARM] Autodetection tests
> 
> On 17/12/15 03:13, Thomas Preud'homme wrote:
> > [Sending on behalf of Andre Vieira]
> >
> > Hi,
> >
> > This patch adds tests for gas' auto detection by checking the build
> attributes assigned to object files. These should correspond to the
> instructions original source as per: 'ARM Architecture Reference Manual
> ARMv7-A and ARMv7-R edition' found at:
> http://infocenter.arm.com/help/topic/com.arm.doc.ddi0406c/index.htm
> l
> >
> > This is correct because for these instructions, the architecture that first
> implements these is represented by the assigned build attributes. There
> are three exceptions here, CLREX, LDREXB and LDREXD. These
> instructions should be assigned attributes to represent ARMv7 and not
> ARMv6T2. The tests were left describing the current behavior, to detect
> any change, potentially pending a fix.
> >
> > Tested by running GAS regressions.
> >
> > Is this OK to commit?
> >
> > Thanks,
> > Andre Vieira
> >
> > gas/testsuite/ChangeLog:
> > 2015-12-09  Andre Vieira  <andre.simoesdiasvieira@arm.com>
> >
> >         * gas/arm/automatic-bw.d: New.
> >         * gas/arm/automatic-bw.s: New.
> >         * gas/arm/automatic-cbz.d: New.
> >         * gas/arm/automatic-cbz.s: New.
> >         * gas/arm/automatic-clrex.d: New.
> >         * gas/arm/automatic-clrex.s: New.
> >         * gas/arm/automatic-lda.d: New.
> >         * gas/arm/automatic-lda.s: New.
> >         * gas/arm/automatic-ldaex.d: New.
> >         * gas/arm/automatic-ldaex.s: New.
> >         * gas/arm/automatic-ldaexb.d: New.
> >         * gas/arm/automatic-ldaexb.s: New.
> >         * gas/arm/automatic-ldrex.d: New.
> >         * gas/arm/automatic-ldrex.s: New.
> >         * gas/arm/automatic-ldrexd.d: New.
> >         * gas/arm/automatic-ldrexd.s: New.
> >         * gas/arm/automatic-movw.d: New.
> >         * gas/arm/automatic-movw.s: New.
> >         * gas/arm/automatic-sdiv.d: New.
> >         * gas/arm/automatic-sdiv.s: New.
> >         * gas/arm/automatic-strexb.d: New.
> >         * gas/arm/automatic-strexb.s: New.
> >
> >
> 
> OK.
> 
> R.