projects / gcc.git / commitdiff
? search:
summary | shortlog | log | commit | commitdiff | tree
raw | patch | inline | side by side (parent: 56941f7)
[arm] Remove FEATURES field from FPU descriptions
authorRichard Earnshaw <rearnsha@arm.com>
Thu, 15 Dec 2016 16:03:06 +0000 (16:03 +0000)
committerRichard Earnshaw <rearnsha@gcc.gnu.org>
Thu, 15 Dec 2016 16:03:06 +0000 (16:03 +0000)
Now that everything uses the new ISA features, we can remove the
FEATURES field from the FPU descriptions, along with all the macros
and definitions associated with it.

* arm-fpus.def (ARM_FPU): Remove features field from all definitions.
* arm.h (arm_fpu_feature_set): Delete typedef.
(FPU_FL_NONE): Delete.
(FPU_FL_NEON): Delete.
(FPU_FL_FP16): Delete.
(FPU_FL_CRYPTO): Delete.
(FPU_FL_DBL): Delete.
(FPU_FL_D32): Delete.
(FPU_FL_VFPv2): Delete.
(FPU_FL_VFPv3): Delete.
(FPU_FL_VFPv4): Delete.
(FPU_FL_VFPv5): Delete.
(FPU_FL_AMRv8): Delete.
(FPU_VFPv2): Delete.
(FPU_VFPv3): Delete.
(FPU_VFPv4): Delete.
(FPU_VFPv5): Delete.
(FPU_ARMv8): Delete.
(FPU_DBL): Delete.
(FPU_D32): Delete.
(FPU_NEON): Delete.
(FPU_CRYPTO): Delete.
(FPU_FP16): Delete.
(arm_fpu_desc): Delete features field.
* arm.c (all_fpus): Don't initialize feature field.

From-SVN: r243715

gcc/ChangeLog patch | blob | history
gcc/config/arm/arm-fpus.def patch | blob | history
gcc/config/arm/arm.c patch | blob | history
gcc/config/arm/arm.h patch | blob | history

diff --git a/gcc/ChangeLog b/gcc/ChangeLog
index 2258bf832166dcfd96d8f4b1e9aad1bba7f2c471..be4e2036a062b72eea9617b300390f7824fe4378 100644 (file)
--- a/gcc/ChangeLog
+++ b/gcc/ChangeLog
@@ -1,3 +1,31 @@
+2016-12-15  Richard Earnshaw  <rearnsha@arm.com>
+
+       * arm-fpus.def (ARM_FPU): Remove features field from all definitions.
+       * arm.h (arm_fpu_feature_set): Delete typedef.
+       (FPU_FL_NONE): Delete.
+       (FPU_FL_NEON): Delete.
+       (FPU_FL_FP16): Delete.
+       (FPU_FL_CRYPTO): Delete.
+       (FPU_FL_DBL): Delete.
+       (FPU_FL_D32): Delete.
+       (FPU_FL_VFPv2): Delete.
+       (FPU_FL_VFPv3): Delete.
+       (FPU_FL_VFPv4): Delete.
+       (FPU_FL_VFPv5): Delete.
+       (FPU_FL_AMRv8): Delete.
+       (FPU_VFPv2): Delete.
+       (FPU_VFPv3): Delete.
+       (FPU_VFPv4): Delete.
+       (FPU_VFPv5): Delete.
+       (FPU_ARMv8): Delete.
+       (FPU_DBL): Delete.
+       (FPU_D32): Delete.
+       (FPU_NEON): Delete.
+       (FPU_CRYPTO): Delete.
+       (FPU_FP16): Delete.
+       (arm_fpu_desc): Delete features field.
+       * arm.c (all_fpus): Don't initialize feature field.
+
 2016-12-15  Richard Earnshaw  <rearnsha@arm.com>
 
        * arm.c (arm_can_inline_p): Use ISA features for determining
diff --git a/gcc/config/arm/arm-fpus.def b/gcc/config/arm/arm-fpus.def
index 1be718fa2cd25dd861b0a5001bd54674ef80b715..ae8197d574933128a0c769309d8a0f7c5623bcb1 100644 (file)
--- a/gcc/config/arm/arm-fpus.def
+++ b/gcc/config/arm/arm-fpus.def
@@ -19,31 +19,31 @@
 
 /* Before using #include to read this file, define a macro:
 
-      ARM_FPU(NAME, ISA, FEATURES)
+      ARM_FPU(NAME, ISA)
 
    The arguments are the fields of struct arm_fpu_desc.
 
    genopt.sh assumes no whitespace up to the first "," in each entry.  */
 
-ARM_FPU("vfp",                 ISA_FEAT(ISA_VFPv2) ISA_FEAT(ISA_FP_DBL),                            FPU_VFPv2 | FPU_DBL)
-ARM_FPU("vfpv2",               ISA_FEAT(ISA_VFPv2) ISA_FEAT(ISA_FP_DBL),                            FPU_VFPv2 | FPU_DBL)
-ARM_FPU("vfpv3",               ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_D32),                            FPU_VFPv3 | FPU_D32)
-ARM_FPU("vfpv3-fp16",          ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_D32) ISA_FEAT(isa_bit_fp16conv), FPU_VFPv3 | FPU_D32 | FPU_FP16)
-ARM_FPU("vfpv3-d16",           ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_DBL),                            FPU_VFPv3 | FPU_DBL)
-ARM_FPU("vfpv3-d16-fp16",      ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_DBL) ISA_FEAT(isa_bit_fp16conv), FPU_VFPv3 | FPU_DBL | FPU_FP16)
-ARM_FPU("vfpv3xd",             ISA_FEAT(ISA_VFPv3),                                                 FPU_VFPv3)
-ARM_FPU("vfpv3xd-fp16",                ISA_FEAT(ISA_VFPv3) ISA_FEAT(isa_bit_fp16conv),                      FPU_VFPv3 | FPU_FP16)
-ARM_FPU("neon",                        ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_NEON),                              FPU_VFPv3 | FPU_NEON)
-ARM_FPU("neon-vfpv3",          ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_NEON),                              FPU_VFPv3 | FPU_NEON)
-ARM_FPU("neon-fp16",           ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_NEON) ISA_FEAT(isa_bit_fp16conv),   FPU_VFPv3 | FPU_NEON | FPU_FP16)
-ARM_FPU("vfpv4",               ISA_FEAT(ISA_VFPv4) ISA_FEAT(ISA_FP_D32),                            FPU_VFPv4 | FPU_D32 | FPU_FP16)
-ARM_FPU("neon-vfpv4",          ISA_FEAT(ISA_VFPv4) ISA_FEAT(ISA_NEON),                              FPU_VFPv4 | FPU_NEON | FPU_FP16)
-ARM_FPU("vfpv4-d16",           ISA_FEAT(ISA_VFPv4) ISA_FEAT(ISA_FP_DBL),                            FPU_VFPv4 | FPU_DBL | FPU_FP16)
-ARM_FPU("fpv4-sp-d16",         ISA_FEAT(ISA_VFPv4),                                                 FPU_VFPv4 | FPU_FP16)
-ARM_FPU("fpv5-sp-d16",         ISA_FEAT(ISA_FPv5),                                                  FPU_VFPv5 | FPU_FP16)
-ARM_FPU("fpv5-d16",            ISA_FEAT(ISA_FPv5) ISA_FEAT(ISA_FP_DBL),                             FPU_VFPv5 | FPU_DBL | FPU_FP16)
-ARM_FPU("fp-armv8",            ISA_FEAT(ISA_FP_ARMv8) ISA_FEAT(ISA_FP_D32),                         FPU_ARMv8 | FPU_D32 | FPU_FP16)
-ARM_FPU("neon-fp-armv8",       ISA_FEAT(ISA_FP_ARMv8) ISA_FEAT(ISA_NEON),                           FPU_ARMv8 | FPU_NEON | FPU_FP16)
-ARM_FPU("crypto-neon-fp-armv8", ISA_FEAT(ISA_FP_ARMv8) ISA_FEAT(ISA_CRYPTO),                        FPU_ARMv8 | FPU_CRYPTO | FPU_FP16)
+ARM_FPU("vfp",                 ISA_FEAT(ISA_VFPv2) ISA_FEAT(ISA_FP_DBL))
+ARM_FPU("vfpv2",               ISA_FEAT(ISA_VFPv2) ISA_FEAT(ISA_FP_DBL))
+ARM_FPU("vfpv3",               ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_D32))
+ARM_FPU("vfpv3-fp16",          ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_D32) ISA_FEAT(isa_bit_fp16conv))
+ARM_FPU("vfpv3-d16",           ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_DBL))
+ARM_FPU("vfpv3-d16-fp16",      ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_DBL) ISA_FEAT(isa_bit_fp16conv))
+ARM_FPU("vfpv3xd",             ISA_FEAT(ISA_VFPv3))
+ARM_FPU("vfpv3xd-fp16",                ISA_FEAT(ISA_VFPv3) ISA_FEAT(isa_bit_fp16conv))
+ARM_FPU("neon",                        ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_NEON))
+ARM_FPU("neon-vfpv3",          ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_NEON))
+ARM_FPU("neon-fp16",           ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_NEON) ISA_FEAT(isa_bit_fp16conv))
+ARM_FPU("vfpv4",               ISA_FEAT(ISA_VFPv4) ISA_FEAT(ISA_FP_D32))
+ARM_FPU("neon-vfpv4",          ISA_FEAT(ISA_VFPv4) ISA_FEAT(ISA_NEON))
+ARM_FPU("vfpv4-d16",           ISA_FEAT(ISA_VFPv4) ISA_FEAT(ISA_FP_DBL))
+ARM_FPU("fpv4-sp-d16",         ISA_FEAT(ISA_VFPv4))
+ARM_FPU("fpv5-sp-d16",         ISA_FEAT(ISA_FPv5))
+ARM_FPU("fpv5-d16",            ISA_FEAT(ISA_FPv5) ISA_FEAT(ISA_FP_DBL))
+ARM_FPU("fp-armv8",            ISA_FEAT(ISA_FP_ARMv8) ISA_FEAT(ISA_FP_D32))
+ARM_FPU("neon-fp-armv8",       ISA_FEAT(ISA_FP_ARMv8) ISA_FEAT(ISA_NEON))
+ARM_FPU("crypto-neon-fp-armv8", ISA_FEAT(ISA_FP_ARMv8) ISA_FEAT(ISA_CRYPTO))
 /* Compatibility aliases.  */
-ARM_FPU("vfp3",                        ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_D32),                            FPU_VFPv3 | FPU_D32)
+ARM_FPU("vfp3",                        ISA_FEAT(ISA_VFPv3) ISA_FEAT(ISA_FP_D32))
diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
index 1d3bb89de7e6e15762cfa326d173c67b9684f7ba..522989d68d09c0063c16e1768359ca241bfe4246 100644 (file)
--- a/gcc/config/arm/arm.c
+++ b/gcc/config/arm/arm.c
@@ -2328,8 +2328,8 @@ char arm_arch_name[] = "__ARM_ARCH_PROFILE__";
 
 const struct arm_fpu_desc all_fpus[] =
 {
-#define ARM_FPU(NAME, ISA, FEATURES)           \
-  { NAME, {ISA isa_nobit}, FEATURES },
+#define ARM_FPU(NAME, ISA)             \
+  { NAME, {ISA isa_nobit} },
 #include "arm-fpus.def"
 #undef ARM_FPU
 };
diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
index 17f030b797a0822b4267f837132150623d015353..4582d2e7f3eee5a32b16fd7bab2c2ba03f17a07e 100644 (file)
--- a/gcc/config/arm/arm.h
+++ b/gcc/config/arm/arm.h
@@ -326,44 +326,10 @@ extern tree arm_fp16_type_node;
   {"mode", "%{!marm:%{!mthumb:-m%(VALUE)}}"}, \
   {"tls", "%{!mtls-dialect=*:-mtls-dialect=%(VALUE)}"},
 
-/* FPU feature sets.  */
-
-typedef unsigned long arm_fpu_feature_set;
-
-/* Test for an FPU feature.  */
-#define ARM_FPU_FSET_HAS(S,F) (((S) & (F)) == (F))
-
-/* FPU Features.  */
-#define FPU_FL_NONE    (0u)
-#define FPU_FL_NEON    (1u << 0)       /* NEON instructions.  */
-#define FPU_FL_FP16    (1u << 1)       /* Half-precision.  */
-#define FPU_FL_CRYPTO  (1u << 2)       /* Crypto extensions.  */
-#define FPU_FL_DBL     (1u << 3)       /* Has double precision.  */
-#define FPU_FL_D32     (1u << 4)       /* Has 32 double precision regs.  */
-#define FPU_FL_VFPv2   (1u << 5)       /* Has VFPv2 features.  */
-#define FPU_FL_VFPv3   (1u << 6)       /* Has VFPv3 extensions.  */
-#define FPU_FL_VFPv4   (1u << 7)       /* Has VFPv4 extensions.  */
-#define FPU_FL_VFPv5   (1u << 8)       /* Has VFPv5 extensions.  */
-#define FPU_FL_ARMv8   (1u << 9)       /* Has ARMv8 extensions to VFP.  */
-
-/* Some useful combinations.  */
-#define FPU_VFPv2      (FPU_FL_VFPv2)
-#define FPU_VFPv3      (FPU_VFPv2 | FPU_FL_VFPv3)
-#define FPU_VFPv4      (FPU_VFPv3 | FPU_FL_VFPv4)
-#define FPU_VFPv5      (FPU_VFPv4 | FPU_FL_VFPv5)
-#define FPU_ARMv8      (FPU_VFPv5 | FPU_FL_ARMv8)
-
-#define FPU_DBL                (FPU_FL_DBL)
-#define FPU_D32                (FPU_DBL | FPU_FL_D32)
-#define FPU_NEON       (FPU_D32 | FPU_FL_NEON)
-#define FPU_CRYPTO     (FPU_NEON | FPU_FL_CRYPTO)
-#define FPU_FP16       (FPU_FL_FP16)
-
 extern const struct arm_fpu_desc
 {
   const char *name;
   enum isa_feature isa_bits[isa_num_bits];
-  arm_fpu_feature_set features;
 } all_fpus[];
 
 /* Which floating point hardware to schedule for.  */