Re: [PATCH] [ARM] Add command line option for RAS extension.

[Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>]

On Fri, Jun 3, 2016 at 9:52 AM, Matthew Wahab
<matthew.wahab@foss.arm.com> wrote:
> Sorry, wrong changelog. Corrected changelog below, patch attached.
> Matthew
>
> On 03/06/16 09:22, Matthew Wahab wrote:
>
>> Support for the RAS extension was added to binutils as part of the
>> ARMv8.2-A architecture, where it is a required feature and is enabled
>> with -march=armv8.2-a. However, the extension can be present as an
>> optional feature in any ARMv8-A architecture.
>>
>> This patch adds the architecture extension "+ras" to enable RAS
>> support. It is enabled by default for -march=armv8.2-a and available but
>> disabled by default for armv8-a and armv8.1-a.
>>
>> Tested for arm-none-linux-gnueabihf with cross-compiled check-binutils,
>> check-gas and check-ld.
>>
>> Ok for trunk?
>> Matthew



>
>
> gas/
> 2016-06-03  Matthew Wahab  <matthew.wahab@arm.com>
>
>         * config/tc-arm.c (arm_ext_v8_2): Rename to arm_ext_ras.
>         (arm_ext_ras): Renamed from arm_ext_v8_2.
>         (insns): Update for arm_ext_v8_2 renaming.
>         (arm_extensions): Add "ras".
>         * doc/c-arm.texi (ARM Options): Add an entry for "ras".
>         * testsuite/gas/arm/armv8-a+ras.d: New.
>         * testsuite/gas/arm/armv8_2-a.d: Add explicit command line
>         options.
>
> include/
> 2016-06-03  Matthew Wahab  <matthew.wahab@arm.com>
>
>         * opcode/arm.h (ARM_EXT2_RAS): New.  Also align preceding
>         entries.
>         (ARM_AEXT_V8_2A): Add ARM_EXT2_RAS.
>
> opcodes/
> 2016-06-03  Matthew Wahab  <matthew.wahab@arm.com>
>
>         * arm-dis.c (arm_opcodes): Replace ARM_EXT_V8_2A with
>         ARM_EXT_RAS in relevant entries.
>

Ok with the fixed changelog. Please apply.


Ramana