From: Ramana Radhakrishnan <ramana.radhakrishnan@foss.arm.com>
To: Matthew Wahab <matthew.wahab@arm.com>
Cc: gcc-patches@gcc.gnu.org
Subject: Re: [PATCH 2/2][ARM] Use new FPU features representation
Date: Mon, 10 Aug 2015 14:59:00 -0000 [thread overview]
Message-ID: <20150810145945.GB32034@e105545-lin> (raw)
In-Reply-To: <5588272C.8060306@arm.com>
On Mon, Jun 22, 2015 at 04:18:04PM +0100, Matthew Wahab wrote:
> Hello,
>
> This patch series changes the representation of FPU features to use a simple
> bit-set and flags, as is done elsewhere.
>
> This patch uses the new representation of FPU feature sets.
>
> Tested the series for arm-none-linux-gnueabihf with check-gcc
>
> Ok for trunk?
> Matthew
This is OK
thanks and sorry about the delay.
ramana
>
> gcc/
> 2015-06-22 Matthew Wahab <matthew.wahab@arm.com>
>
> * config/arm/arm-fpus.def: Replace neon, fp16 and crypto boolean
> fields with feature flags. Update comment.
> * config/arm/arm.c (ARM_FPU): Update macro.
> * config/arm/arm.h (TARGET_NEON_FP16): Update feature test.
> (TARGET_FP16): Likewise.
> (TARGET_CRYPTO): Likewise.
> (TARGET_NEON): Likewise.
> (struct arm_fpu_desc): Remove fields neon, fp16 and crypto. Add
> field features.
>
> From 6f9cd1b41d7597d95bd80aa21344f8e6e011e168 Mon Sep 17 00:00:00 2001
> From: Matthew Wahab <matthew.wahab@arm.com>
> Date: Wed, 10 Jun 2015 10:11:56 +0100
> Subject: [PATCH 2/2] Use new FPU feature definitions.
>
> Change-Id: I0c45e52b08b31433ec2b30fcb666584cabcb826b
> ---
> gcc/config/arm/arm-fpus.def | 40 ++++++++++++++++++++--------------------
> gcc/config/arm/arm.c | 4 ++--
> gcc/config/arm/arm.h | 22 +++++++++++++---------
> 3 files changed, 35 insertions(+), 31 deletions(-)
>
> diff --git a/gcc/config/arm/arm-fpus.def b/gcc/config/arm/arm-fpus.def
> index 2dfefd6..efd5896 100644
> --- a/gcc/config/arm/arm-fpus.def
> +++ b/gcc/config/arm/arm-fpus.def
> @@ -19,30 +19,30 @@
>
> /* Before using #include to read this file, define a macro:
>
> - ARM_FPU(NAME, MODEL, REV, VFP_REGS, NEON, FP16, CRYPTO)
> + ARM_FPU(NAME, MODEL, REV, VFP_REGS, FEATURES)
>
> 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", ARM_FP_MODEL_VFP, 2, VFP_REG_D16, false, false, false)
> -ARM_FPU("vfpv3", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, false, false)
> -ARM_FPU("vfpv3-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, true, false)
> -ARM_FPU("vfpv3-d16", ARM_FP_MODEL_VFP, 3, VFP_REG_D16, false, false, false)
> -ARM_FPU("vfpv3-d16-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_D16, false, true, false)
> -ARM_FPU("vfpv3xd", ARM_FP_MODEL_VFP, 3, VFP_REG_SINGLE, false, false, false)
> -ARM_FPU("vfpv3xd-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_SINGLE, false, true, false)
> -ARM_FPU("neon", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, true , false, false)
> -ARM_FPU("neon-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, true, true, false)
> -ARM_FPU("vfpv4", ARM_FP_MODEL_VFP, 4, VFP_REG_D32, false, true, false)
> -ARM_FPU("vfpv4-d16", ARM_FP_MODEL_VFP, 4, VFP_REG_D16, false, true, false)
> -ARM_FPU("fpv4-sp-d16", ARM_FP_MODEL_VFP, 4, VFP_REG_SINGLE, false, true, false)
> -ARM_FPU("fpv5-sp-d16", ARM_FP_MODEL_VFP, 5, VFP_REG_SINGLE, false, true, false)
> -ARM_FPU("fpv5-d16", ARM_FP_MODEL_VFP, 5, VFP_REG_D16, false, true, false)
> -ARM_FPU("neon-vfpv4", ARM_FP_MODEL_VFP, 4, VFP_REG_D32, true, true, false)
> -ARM_FPU("fp-armv8", ARM_FP_MODEL_VFP, 8, VFP_REG_D32, false, true, false)
> -ARM_FPU("neon-fp-armv8",ARM_FP_MODEL_VFP, 8, VFP_REG_D32, true, true, false)
> +ARM_FPU("vfp", ARM_FP_MODEL_VFP, 2, VFP_REG_D16, FPU_FL_NONE)
> +ARM_FPU("vfpv3", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, FPU_FL_NONE)
> +ARM_FPU("vfpv3-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, FPU_FL_FP16)
> +ARM_FPU("vfpv3-d16", ARM_FP_MODEL_VFP, 3, VFP_REG_D16, FPU_FL_NONE)
> +ARM_FPU("vfpv3-d16-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_D16, FPU_FL_FP16)
> +ARM_FPU("vfpv3xd", ARM_FP_MODEL_VFP, 3, VFP_REG_SINGLE, FPU_FL_NONE)
> +ARM_FPU("vfpv3xd-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_SINGLE, FPU_FL_FP16)
> +ARM_FPU("neon", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, FPU_FL_NEON)
> +ARM_FPU("neon-fp16", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, FPU_FL_NEON | FPU_FL_FP16)
> +ARM_FPU("vfpv4", ARM_FP_MODEL_VFP, 4, VFP_REG_D32, FPU_FL_FP16)
> +ARM_FPU("vfpv4-d16", ARM_FP_MODEL_VFP, 4, VFP_REG_D16, FPU_FL_FP16)
> +ARM_FPU("fpv4-sp-d16", ARM_FP_MODEL_VFP, 4, VFP_REG_SINGLE, FPU_FL_FP16)
> +ARM_FPU("fpv5-sp-d16", ARM_FP_MODEL_VFP, 5, VFP_REG_SINGLE, FPU_FL_FP16)
> +ARM_FPU("fpv5-d16", ARM_FP_MODEL_VFP, 5, VFP_REG_D16, FPU_FL_FP16)
> +ARM_FPU("neon-vfpv4", ARM_FP_MODEL_VFP, 4, VFP_REG_D32, FPU_FL_NEON | FPU_FL_FP16)
> +ARM_FPU("fp-armv8", ARM_FP_MODEL_VFP, 8, VFP_REG_D32, FPU_FL_FP16)
> +ARM_FPU("neon-fp-armv8",ARM_FP_MODEL_VFP, 8, VFP_REG_D32, FPU_FL_NEON | FPU_FL_FP16)
> ARM_FPU("crypto-neon-fp-armv8",
> - ARM_FP_MODEL_VFP, 8, VFP_REG_D32, true, true, true)
> + ARM_FP_MODEL_VFP, 8, VFP_REG_D32, FPU_FL_NEON | FPU_FL_FP16 | FPU_FL_CRYPTO)
> /* Compatibility aliases. */
> -ARM_FPU("vfp3", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, false, false, false)
> +ARM_FPU("vfp3", ARM_FP_MODEL_VFP, 3, VFP_REG_D32, FPU_FL_NONE)
> diff --git a/gcc/config/arm/arm.c b/gcc/config/arm/arm.c
> index e79a369..e104d2f 100644
> --- a/gcc/config/arm/arm.c
> +++ b/gcc/config/arm/arm.c
> @@ -2231,8 +2231,8 @@ char arm_arch_name[] = "__ARM_ARCH_0UNK__";
>
> static const struct arm_fpu_desc all_fpus[] =
> {
> -#define ARM_FPU(NAME, MODEL, REV, VFP_REGS, NEON, FP16, CRYPTO) \
> - { NAME, MODEL, REV, VFP_REGS, NEON, FP16, CRYPTO },
> +#define ARM_FPU(NAME, MODEL, REV, VFP_REGS, FEATURES) \
> + { NAME, MODEL, REV, VFP_REGS, FEATURES },
> #include "arm-fpus.def"
> #undef ARM_FPU
> };
> diff --git a/gcc/config/arm/arm.h b/gcc/config/arm/arm.h
> index eadbcec..26d48a7 100644
> --- a/gcc/config/arm/arm.h
> +++ b/gcc/config/arm/arm.h
> @@ -193,11 +193,13 @@ extern void (*arm_lang_output_object_attributes_hook)(void);
> #define TARGET_VFP_DOUBLE (TARGET_VFP && arm_fpu_desc->regs != VFP_REG_SINGLE)
>
> /* FPU supports half-precision floating-point with NEON element load/store. */
> -#define TARGET_NEON_FP16 \
> - (TARGET_VFP && arm_fpu_desc->neon && arm_fpu_desc->fp16)
> +#define TARGET_NEON_FP16 \
> + (TARGET_VFP \
> + && ARM_FPU_FSET_HAS (arm_fpu_desc->features, FPU_FL_NEON | FPU_FL_FP16))
>
> /* FPU supports VFP half-precision floating-point. */
> -#define TARGET_FP16 (TARGET_VFP && arm_fpu_desc->fp16)
> +#define TARGET_FP16 \
> + (TARGET_VFP && ARM_FPU_FSET_HAS (arm_fpu_desc->features, FPU_FL_FP16))
>
> /* FPU supports fused-multiply-add operations. */
> #define TARGET_FMA (TARGET_VFP && arm_fpu_desc->rev >= 4)
> @@ -206,14 +208,18 @@ extern void (*arm_lang_output_object_attributes_hook)(void);
> #define TARGET_FPU_ARMV8 (TARGET_VFP && arm_fpu_desc->rev >= 8)
>
> /* FPU supports Crypto extensions. */
> -#define TARGET_CRYPTO (TARGET_VFP && arm_fpu_desc->crypto)
> +#define TARGET_CRYPTO \
> + (TARGET_VFP && ARM_FPU_FSET_HAS (arm_fpu_desc->features, FPU_FL_CRYPTO))
> +
>
> /* FPU supports Neon instructions. The setting of this macro gets
> revealed via __ARM_NEON__ so we add extra guards upon TARGET_32BIT
> and TARGET_HARD_FLOAT to ensure that NEON instructions are
> available. */
> -#define TARGET_NEON (TARGET_32BIT && TARGET_HARD_FLOAT \
> - && TARGET_VFP && arm_fpu_desc->neon)
> +#define TARGET_NEON \
> + (TARGET_32BIT && TARGET_HARD_FLOAT && TARGET_VFP \
> + && ARM_FPU_FSET_HAS (arm_fpu_desc->features, FPU_FL_NEON))
> +
>
> /* Q-bit is present. */
> #define TARGET_ARM_QBIT_P(flags) \
> @@ -353,9 +359,7 @@ extern const struct arm_fpu_desc
> enum arm_fp_model model;
> int rev;
> enum vfp_reg_type regs;
> - int neon;
> - int fp16;
> - int crypto;
> + arm_fpu_fset features;
> } *arm_fpu_desc;
>
> /* Which floating point hardware to schedule for. */
> --
> 1.9.1
>
next prev parent reply other threads:[~2015-08-10 14:59 UTC|newest]
Thread overview: 7+ messages / expand[flat|nested] mbox.gz Atom feed top
2015-06-22 15:17 [PATCH 1/2][ARM] Record FPU features as a bit-set Matthew Wahab
2015-06-22 15:18 ` [PATCH 2/2][ARM] Use new FPU features representation Matthew Wahab
2015-08-10 10:28 ` Matthew Wahab
2015-08-10 14:59 ` Ramana Radhakrishnan [this message]
2015-08-10 10:28 ` [PATCH 1/2][ARM] Record FPU features as a bit-set Matthew Wahab
2015-08-10 10:55 ` Kyrill Tkachov
2015-08-10 14:57 ` Ramana Radhakrishnan
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20150810145945.GB32034@e105545-lin \
--to=ramana.radhakrishnan@foss.arm.com \
--cc=gcc-patches@gcc.gnu.org \
--cc=matthew.wahab@arm.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).