This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64] ARMv8.2 command line and feature macros support
- From: Andrew Pinski <pinskia at gmail dot com>
- To: Jiong Wang <jiong dot wang at foss dot arm dot com>
- Cc: GCC Patches <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 29 Jun 2016 14:24:25 -0700
- Subject: Re: [AArch64] ARMv8.2 command line and feature macros support
- Authentication-results: sourceware.org; auth=none
- References: <8fd20f47-7f86-0bec-be43-87903502d696 at foss dot arm dot com> <39254e41-2963-bf5a-a359-41a64dc0701e at foss dot arm dot com>
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".
>
>