[AArch64] ARMv8.2 command line and feature macros support

[Andrew Pinski]

On Mon, Jun 27, 2016 at 7:58 AM, Jiong Wang <jiong.wang@foss.arm.com> wrote:
> On 07/06/16 09:46, Jiong Wang wrote:
>>
>> 2016-06-07  Matthew Wahab<matthew.wahab@arm.com>
>>             Jiong Wang<jiong.wang@arm.com>
>>
>>         * config/aarch64/aarch64-arches.def: Add "armv8.2-a".
>>         * config/aarch64/aarch64.h (AARCH64_FL_V8_2): New.
>>         (AARCH64_FL_F16): New.
>>         (AARCH64_FL_FOR_ARCH8_2): New.
>>         (AARCH64_ISA_8_2): New.
>>         (AARCH64_ISA_F16): New.
>>         (TARGET_FP_F16INST): New.
>>         (TARGET_SIMD_F16INST): New.
>>         * config/aarch64/aarch64-option-extensions.def: New entry for
>> "fp16".
>>         * config/aarch64/aarch64-c.c (arch64_update_cpp_builtins):
>> Conditionally define
>>         __ARM_FEATURE_FP16_SCALAR_ARITHMETIC and
>> __ARM_FEATURE_FP16_VECTOR_ARITHMETIC.
>>         * doc/invoke.texi (AArch64 Options): Document "armv8.2-a" and
>> "fp16".
>>
>
> This is a updated version of this patch, the updates are:
>
>   * When enabling "fp16" also enables "fp".
>   * When disabling "fp" also disables "fp16".
>   * When disabling "fp16" only disables "fp16".
>
> OK for trunk?

I notice you did not add a profile entry in
aarch64-option-extensions.def.  Are you going to add this separately?
People will use gcc to invoke the assembler and use
-mcpu=armv8.2-a+profile on the command line.  In fact I already got a
request internally to support that.

Thanks,
Andrew

>
> 2016-06-27  Matthew Wahab  <matthew.wahab@arm.com>
>             Jiong Wang  <jiong.wang@arm.com>
>
>         * config/aarch64/aarch64-arches.def: Add "armv8.2-a".
>         * config/aarch64/aarch64.h (AARCH64_FL_V8_2): New.
>         (AARCH64_FL_F16): New.
>         (AARCH64_FL_FOR_ARCH8_2): New.
>         (AARCH64_ISA_8_2): New.
>         (AARCH64_ISA_F16): New.
>         (TARGET_FP_F16INST): New.
>         (TARGET_SIMD_F16INST): New.
>         * config/aarch64/aarch64-option-extensions.def ("fp16"): New entry.
>         ("fp"): Disabling "fp" also disables "fp16".
>
>         * config/aarch64/aarch64-c.c (arch64_update_cpp_builtins):
> Conditionally define
>         __ARM_FEATURE_FP16_SCALAR_ARITHMETIC and
> __ARM_FEATURE_FP16_VECTOR_ARITHMETIC.
>         * doc/invoke.texi (AArch64 Options): Document "armv8.2-a" and
> "fp16".
>
>