On Mon, Jun 27, 2016 at 7:58 AM, Jiong Wang <jiong.w...@foss.arm.com> wrote:
> On 07/06/16 09:46, Jiong Wang wrote:
>>
>> 2016-06-07 Matthew Wahab<matthew.wa...@arm.com>
>> Jiong Wang<jiong.w...@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.wa...@arm.com>
> Jiong Wang <jiong.w...@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".
>
>
