Message ID | 9196175c-df7d-8cf1-b95e-8722d98737b1@arm.com |
---|---|
State | New |
Headers | show |
Series | [aarch64] Enable FP16 feature by default for Armv9 | expand |
> -----Original Message----- > From: Andre Vieira (lists) <andre.simoesdiasvieira@arm.com> > Sent: Tuesday, March 8, 2022 3:20 PM > To: gcc-patches@gcc.gnu.org > Cc: Richard Sandiford <Richard.Sandiford@arm.com>; Kyrylo Tkachov > <Kyrylo.Tkachov@arm.com> > 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 <andre.simoesdiasvieira@arm.com> > > * 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. */