This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH, ARM 7/7] Add support for ARMv8-M Mainline with DSP extension
- From: Thomas Preudhomme <thomas dot preudhomme at foss dot arm dot com>
- To: Nick Clifton <nickc at redhat dot com>
- Cc: binutils at sourceware dot org
- Date: Tue, 12 Apr 2016 15:18:57 +0100
- Subject: Re: [PATCH, ARM 7/7] Add support for ARMv8-M Mainline with DSP extension
- Authentication-results: sourceware.org; auth=none
- References: <004401d13d50$48322960$d8967c20$ at foss dot arm dot com> <4090595 dot 4ZTEnnfHGC at e108577-lin> <56FBF040 dot 8070007 at redhat dot com>
On Wednesday 30 March 2016 16:26:56 Nick Clifton wrote:
> Hi Thomas,
>
> > *** bfd/ChangeLog ***
> > 2015-11-10 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > (elf32_arm_merge_eabi_attributes): Add merging logic for
> > Tag_DSP_extension.
> >
> > *** binutils/ChangeLog ***
> > 2015-11-10 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > * readelf.c (display_arm_attribute): Add output for
> > Tag_DSP_extension.
> > (arm_attr_public_tags): Define DSP_extension attribute.
> >
> > *** gas/ChangeLog ***
> > 2016-03-07 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > * config/tc-arm.c (arm_ext_dsp): New feature for Thumb DSP
> > instructions.
> > (arm_extensions): Add dsp extension for ARMv8-M Mainline.
> > (aeabi_set_public_attributes): Memorize the feature bits of the
> > architecture selected for Tag_CPU_arch. Use it to set
> > Tag_DSP_extension to 1 for ARMv8-M Mainline with DSP extension.
> > (arm_convert_symbolic_attribute): Define Tag_DSP_extension.
> > * testsuite/gas/arm/arch7em-bad.d: Rename to ...
> > * testsuite/gas/arm/arch7em-bad-1.d: This.
> > * testsuite/gas/arm/arch7em-bad-2.d: New file.
> > * testsuite/gas/arm/arch7em-bad-3.d: Likewise.
> > * testsuite/gas/arm/archv8m-main-dsp-1.d: Likewise.
> > * testsuite/gas/arm/archv8m-main-dsp-2.d: Likewise.
> > * testsuite/gas/arm/archv8m-main-dsp-3.d: Likewise.
> > * testsuite/gas/arm/archv8m-main-dsp-4.d: Likewise.
> > * testsuite/gas/arm/archv8m-main-dsp-5.d: Likewise.
> > * testsuite/gas/arm/attr-march-armv8m.main.dsp.d: Likewise.
> >
> > *** include/elf/ChangeLog ***
> > 2015-11-10 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > * arm.h (Tag_DSP_extension): Define.
> >
> > *** ld/ChangeLog ***
> > 2016-03-07 Thomas Preud'homme <thomas.preudhomme@arm.com>
> >
> > * testsuite/ld-arm/arm-elf.exp (EABI attribute merging 10 (DSP)):
> > New
> > test.
> > * testsuite/ld-arm/attr-merge-10b-dsp.s: New file.
> > * testsuite/ld-arm/attr-merge-10-dsp.attr: Likewise.
>
> Approved - please apply - but ...
>
> Please could you also add an entry to gas/NEWS mentioning the support for
> the new instructions.
I'm a bit confused at the current content of gas/NEWS. It mentions twice
"Changes in 2.26". Is the top one supposed to be 2.27? I'm not sure about it
because it has been created on November 13th of last year.
Best regards,
Thomas