Hi all,

On 9/10/19 1:34 PM, Stam Markianos-Wright wrote:

Hi all,

This is a minor patch that fixes the entry for the fp16fml feature in
GCC's aarch64-option-extensions.def.

As can be seen in the Linux sources here
https://github.com/torvalds/linux/blob/master/arch/arm64/kernel/cpuinfo.c#L69
the correct string is "asimdfhm", not "asimdfml".

Cross-compiled and tested on aarch64-none-linux-gnu.

Is this ok for trunk?

Also, I don't have commit rights, so could someone commit it on my behalf?

James approved it offline so I've committed it on Stam's behalf as r276097 with a slightly adjusted ChangeLog:

2019-09-24  Stamatis Markianos-Wright <stam.markianos-wri...@arm.com>

    * config/aarch64/aarch64-option-extensions.def (fp16fml):
    Update hwcap string for fp16fml.

Thanks,

Kyrill

Thanks,
Stam Markianos-Wright


The diff is:

diff --git a/gcc/config/aarch64/aarch64-option-extensions.def
b/gcc/config/aarch64/aarch64-option-extensions.def
index 9919edd43d0..60e8f28fff5 100644
--- a/gcc/config/aarch64/aarch64-option-extensions.def
+++ b/gcc/config/aarch64/aarch64-option-extensions.def
@@ -135,7 +135,7 @@ AARCH64_OPT_EXTENSION("sm4", AARCH64_FL_SM4,
AARCH64_FL_SIMD, \
   /* Enabling "fp16fml" also enables "fp" and "fp16".
      Disabling "fp16fml" just disables "fp16fml".  */
   AARCH64_OPT_EXTENSION("fp16fml", AARCH64_FL_F16FML, \
-              AARCH64_FL_FP | AARCH64_FL_F16, 0, false, "asimdfml")
+              AARCH64_FL_FP | AARCH64_FL_F16, 0, false, "asimdfhm")

   /* Enabling "sve" also enables "fp16", "fp" and "simd".
      Disabling "sve" disables "sve", "sve2", "sve2-aes", "sve2-sha3",
"sve2-sm4"



gcc/ChangeLog:

2019-09-09  Stamatis Markianos-Wright <stam.markianos-wri...@arm.com>

      * config/aarch64/aarch64-option-extensions.def: Updated hwcap
string for fp16fml.


Reply via email to