RE: [aarch64] Enable FP16 feature by default for Armv9

2022-03-08 Thread Kyrylo Tkachov via Gcc-patches


> -Original Message-
> From: Andre Vieira (lists) 
> Sent: Tuesday, March 8, 2022 3:20 PM
> To: gcc-patches@gcc.gnu.org
> Cc: Richard Sandiford ; Kyrylo Tkachov
> 
> Subject: [aarch64] Enable FP16 feature by default for Armv9
> 
> Hi all,
> 
> This patch adds the feature bit for FP16 to the feature set for Armv9
> since Armv9 requires SVE to be implemented and SVE requires FP16 to be
> implemented.

Ok.
Thanks,
Kyrill

P.S. We may want to updated the Neoverse N2 entry in aarch64-cores.def to use 
AARCH64_FL_FOR_ARCH9. That entry was added before AARCH64_FL_FOR_ARCH9 so it 
uses the old v8.5-based flag.

> 
> 2022-03-04  Andre Vieira  
> 
>      * config/aarch64/aarch64.h (AARCH64_FL_FOR_ARCH9): Add FP16
> feature bit.


[aarch64] Enable FP16 feature by default for Armv9

2022-03-08 Thread Andre Vieira (lists) via Gcc-patches

Hi all,

This patch adds the feature bit for FP16 to the feature set for Armv9 
since Armv9 requires SVE to be implemented and SVE requires FP16 to be 
implemented.


2022-03-04  Andre Vieira  

    * config/aarch64/aarch64.h (AARCH64_FL_FOR_ARCH9): Add FP16 
feature bit.
diff --git a/gcc/config/aarch64/aarch64.h b/gcc/config/aarch64/aarch64.h
index 
27ba4f4ca3fa78585733cfe68e2dee32c55282a7..efa46ac0b8799b5849b609d591186e26e5cb37ff
 100644
--- a/gcc/config/aarch64/aarch64.h
+++ b/gcc/config/aarch64/aarch64.h
@@ -278,7 +278,8 @@ extern unsigned aarch64_architecture_version;
 #define AARCH64_FL_FOR_ARCH8_R \
   (AARCH64_FL_FOR_ARCH8_4 | AARCH64_FL_V8_R)
 #define AARCH64_FL_FOR_ARCH9   \
-  (AARCH64_FL_FOR_ARCH8_5 | AARCH64_FL_SVE | AARCH64_FL_SVE2 | AARCH64_FL_V9)
+  (AARCH64_FL_FOR_ARCH8_5 | AARCH64_FL_SVE | AARCH64_FL_SVE2 | AARCH64_FL_V9 \
+   | AARCH64_FL_F16)
 
 /* Macros to test ISA flags.  */