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?

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.


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"

Reply via email to