This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH, BINUTILS, AARCH64, 2/9] Add Data procoessing instructions for ARMv8.5-A
- From: "Richard Earnshaw (lists)" <Richard dot Earnshaw at arm dot com>
- To: Sudakshina Das <sudi dot das at arm dot com>, "binutils at sourceware dot org" <binutils at sourceware dot org>
- Cc: nd <nd at arm dot com>, Ramana Radhakrishnan <Ramana dot Radhakrishnan at arm dot com>, Nick Clifton <nickc at redhat dot com>
- Date: Tue, 9 Oct 2018 15:42:11 +0100
- Subject: Re: [PATCH, BINUTILS, AARCH64, 2/9] Add Data procoessing instructions for ARMv8.5-A
- References: <525b94be-0a36-0ccf-68db-844c11f91837@arm.com>
On 19/09/18 15:17, Sudakshina Das wrote:
> Hi
>
> This patch is part of the patch series to add support for ARMv8.5-A
> extensions.
> (https://developer.arm.com/products/architecture/cpu-architecture/a-profile/docs/ddi0596/a/a64-base-instructions-alphabetic-order)
>
>
> This patch adds the data processing instructions that are new to
> ARMv8.5-A.
>
> 1) There are 2 instructions: xaflag, axflag, that are added
> to manipulate the states of the flag and are used to convert
> between the Arm representation and the fcmp representation.
>
> 2) The other instructions are rounding instructions which have
> 8 versions based on whether the floating-point number is a
> Single-Precision or Double-Precision number, whether the
> target integer is a 32-bit or 64-bit integer and whether the
> rounding mode is the ambient rounding mode or to zero. Each of
> these instruction is available in both Scalar and Vector forms.
>
> Since both 1) and 2) have separate identification mechanism and
> it is permissible that a ARMv8.4 compliant implementation may include
> any arbitrary subset of the ARMv8.5 features unless otherwise
> specified, new feature bits are added.
>
> Testing done: Builds and reg tests all pass on aarch64-none-linux-gnu
> and aarch64-none-elf. Added new test.
>
> Thanks
> Sudi
> PS. I do not have commit access so if OK can someone apply for me?
> Also the patch was too big to attach due to the regenerated files.
>
> *** include/ChangeLog ***
>
> 2018-xx-xx Sudakshina Das <sudi.das@arm.com>
>
> * opcode/aarch64.h (AARCH64_FEATURE_FLAGMANIP): New.
> (AARCH64_FEATURE_FRINTTS): New.
> (AARCH64_ARCH_V8_5): Add both by default.
>
> *** opcodes/ChangeLog ***
>
> 2018-xx-xx Sudakshina Das <sudi.das@arm.com>
>
> * aarch64-tbl.h (aarch64_feature_flagmanip): New.
> (aarch64_feature_frintts): New.
> (FLAGMANIP, FRINTTS): New.
> (aarch64_opcode_table): Add entries for xaflag, axflag
> and frint[32,64][x,z] instructions.
> * aarch64-asm-2.c: Regenerate.
> * aarch64-dis-2.c: Regenerate.
> * aarch64-opc-2.c: Regenerate.
>
> *** gas/ChangeLog ***
>
> 2018-xx-xx Sudakshina Das <sudi.das@arm.com>
>
> * testsuite/gas/aarch64/armv8_5-a-dp.s: New.
> * testsuite/gas/aarch64/armv8_5-a-dp.d: New.
Committed.
R.