[PATCH][GAS][AArch64] Move new FP16 instructions to new flag

tamar.christina@arm.com tamar.christina@arm.com
Wed Nov 15 12:47:00 GMT 2017


Hi All,

As was pointed out to fp16 is not mandatory on any architecture,

So I have modified this patch accordingly.

Regtested on aarch64-none-elf and no issues.

Ok for master?
Thanks,
Tamar

gas/
2017-11-15  Tamar Christina  <tamar.christina@arm.com>

	* config/tc-aarch64.c (fp16fml): New.
	* doc/c-aarch64.texi (fp16fml): New.
	* testsuite/gas/aarch64/armv8_2-a-crypto-fp16.d (fp16): Make fp16fml.
	* testsuite/gas/aarch64/armv8_3-a-crypto-fp16.d (fp16): Make fp16fml.
	

include/
2017-11-15  Tamar Christina  <tamar.christina@arm.com>

	* opcode/aarch64.h: (AARCH64_FEATURE_F16_FML): New.
	(AARCH64_ARCH_V8_4): Enable AARCH64_FEATURE_F16_FML by default.

opcodes/
2017-11-15  Tamar Christina  <tamar.christina@arm.com>

	* aarch64-tbl.h (aarch64_feature_fp_16_v8_2): Require AARCH64_FEATURE_F16_FML
	and AARCH64_FEATURE_F16.

> -----Original Message-----
> From: binutils-owner@sourceware.org [mailto:binutils-
> owner@sourceware.org] On Behalf Of Tamar Christina
> Sent: Wednesday, November 15, 2017 11:51
> To: binutils@sourceware.org
> Cc: nd <nd@arm.com>; Richard Earnshaw <Richard.Earnshaw@arm.com>;
> Marcus Shawcroft <Marcus.Shawcroft@arm.com>
> Subject: [PATCH][GAS][AArch64] Move new FP16 instructions to new flag
> 
> Hi All,
> 
> This patch separates the new FP16 instructions backported from Armv8.4-a
> to Armv8.2-a into a new flag order to distinguish them from the rest of the
> already existing optional
> FP16 instructions in Armv8.2-a.
> 
> The new flag "+fp16fml" is available from Armv8.2-a and implies +fp16 and is
> mandatory on Armv8.4-a.
> 
> Regtested on aarch64-none-elf and no regressions.
> 
> Ok for master?
> 
> Thanks,
> Tamar
> 
> gas/
> 2017-11-15  Tamar Christina  <tamar.christina@arm.com>
> 
> 	* config/tc-aarch64.c (fp16fml): New.
> 	* doc/c-aarch64.texi (fp16fml): New.
> 	* testsuite/gas/aarch64/armv8_2-a-crypto-fp16.d (fp16): Make
> fp16fml.
> 	* testsuite/gas/aarch64/armv8_3-a-crypto-fp16.d (fp16): Make
> fp16fml.
> 	* testsuite/gas/aarch64/armv8_4-a-crypto-fp16.d (fp16): Remove
> +fp16.
> 
> 
> include/
> 2017-11-15  Tamar Christina  <tamar.christina@arm.com>
> 
> 	* opcode/aarch64.h: (AARCH64_FEATURE_F16_FML): New.
> 	(AARCH64_ARCH_V8_2): Enable AARCH64_FEATURE_F16 by default.
> 	(AARCH64_ARCH_V8_4): Enable AARCH64_FEATURE_F16_FML by
> default.
> 
> opcodes/
> 2017-11-15  Tamar Christina  <tamar.christina@arm.com>
> 
> 	* aarch64-tbl.h (aarch64_feature_fp_16_v8_2): Use
> AARCH64_FEATURE_F16_FML.
> 
> --
-------------- next part --------------
A non-text attachment was scrubbed...
Name: fp16fml-aarch64.patch
Type: text/x-diff
Size: 4238 bytes
Desc: not available
URL: <https://sourceware.org/pipermail/binutils/attachments/20171115/d52a9894/attachment.bin>


More information about the Binutils mailing list