This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH][binutils][arm] PR25376 Change MVE into a CORE_HIGH feature
- From: Nick Clifton <nickc at redhat dot com>
- To: "Andre Vieira (lists)" <andre dot simoesdiasvieira at arm dot com>, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Cc: Richard Earnshaw <Richard dot Earnshaw at arm dot com>
- Date: Wed, 15 Jan 2020 14:56:45 +0000
- Subject: Re: [PATCH][binutils][arm] PR25376 Change MVE into a CORE_HIGH feature
- References: <8e5b74b6-1d6f-b7cb-5271-c204a4f6a500@arm.com>
Hi Andre,
> gas/ChangeLog:
> 2020-01-13 Andre Vieira <andre.simoesdiasvieira@arm.com>
>
> PR 25376
> * config/tc-arm.c (mve_ext, mve_fp_ext): Use CORE_HIGH.
> (armv8_1m_main_ext_table): Use CORE_HIGH for mve.
> * testsuite/arm/armv8_1-m-fpu-mve-1.s: New.
> * testsuite/arm/armv8_1-m-fpu-mve-1.d: New.
> * testsuite/arm/armv8_1-m-fpu-mve-2.s: New.
> * testsuite/arm/armv8_1-m-fpu-mve-2.d: New.
>
>
> include/ChangeLog:
> 2020-01-13 Andre Vieira <andre.simoesdiasvieira@arm.com>
>
> PR 25376
> * opcodes/arm.h (FPU_MVE, FPU_MVE_FPU): Move these features to...
> (ARM_EXT2_MVE, ARM_EXT2_MVE_FP): ... the CORE_HIGH space.
> (ARM_ANY): Redefine to not include any MVE bits.
> (ARM_FEATURE_ALL): Removed.
>
> opcodes/ChangeLog:
> 2020-01-13 Andre Vieira <andre.simoesdiasvieira@arm.com>
>
> PR 25376
> * opcodes/arm-dis.c (coprocessor_opcodes): Use CORE_HIGH for
> MVE bits.
> (neon_opcodes): Likewise.
> (select_arm_features): Make sure we enable MVE bits when
> selecting armv8.1-m.main. Make sure we do not enable MVE
> bits when not selecting any architecture.
Approved - please apply.
Cheers
Nick