This is the mail archive of the
binutils@sourceware.org
mailing list for the binutils project.
Re: [PATCH, BINUTILS, AARCH64, 3/9] Add instruction SB 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>, Nick Clifton <nickc at redhat dot com>
- Date: Tue, 9 Oct 2018 15:42:28 +0100
- Subject: Re: [PATCH, BINUTILS, AARCH64, 3/9] Add instruction SB for ARMv8.5-A
- References: <2013088a-af57-601b-7079-66d539a7ec71@arm.com>
On 19/09/18 15:19, 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 instruction is retrospectively made optional for all versions of
> the architecture from ARMv8.0 to ARMv8.4 and is mandatory from ARMv8.5.
> Hence a new command line option of "+sb" is added for older
> architectures.
>
> Testing done: Builds and reg tests all pass on aarch64-none-linux-gnu
> and aarch64-none-elf. Added new tests.
>
> 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_SB): New.
> (AARCH64_ARCH_V8_5): Add AARCH64_FEATURE_SB by default.
>
> *** opcodes/ChangeLog ***
>
> 2018-xx-xx Sudakshina Das <sudi.das@arm.com>
>
> * aarch64-tbl.h (aarch64_feature_sb): New.
> (SB, SB_INSN): New.
> (aarch64_opcode_table): Add entry for sb.
> * 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>
>
> * config/tc-aarch64.c (aarch64_features): Add new "sb" option
> for older architectures.
> * doc/c-aarch64.texi: Document the same.
> * testsuite/gas/aarch64/sb.s: New.
> * testsuite/gas/aarch64/sb.d: New.
>
>
>
Committed.
R.