This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH 4/5][ARM] Use features sets for builtins.
- From: Ramana Radhakrishnan <ramana dot gcc at googlemail dot com>
- To: Matthew Wahab <matthew dot wahab at foss dot arm dot com>
- Cc: gcc-patches <gcc-patches at gcc dot gnu dot org>
- Date: Tue, 18 Aug 2015 08:27:52 +0100
- Subject: Re: [PATCH 4/5][ARM] Use features sets for builtins.
- Authentication-results: sourceware.org; auth=none
- References: <55C89141 dot 8050304 at foss dot arm dot com> <55C89202 dot 4030602 at foss dot arm dot com>
- Reply-to: ramrad01 at arm dot com
On Mon, Aug 10, 2015 at 12:58 PM, Matthew Wahab
<matthew.wahab@foss.arm.com> wrote:
> The ARM backend uses an unsigned long to record CPU feature flags and
> there are currently 31 bits in use. This series of patches replaces the
> single unsigned long with a representation based on an array of values.
>
> This patch updates the feature flags usage in the builtins description
> to be able use make of all representable flags.
>
> Tested the series for arm-none-linux-gnueabihf with native bootstrap and
> make check.
>
> gcc/
> 2015-08-10 Matthew Wahab <matthew.wahab@arm.com>
>
> * config/arm/arm-builtins.c (def_mbuiltin): Test all flags in a
> feature set.
> (struct builtin_description): Replace field mask with field
> features.
> (IWMMXT_BUILTIN): Use ARM_FSET macros for feature flags.
> (IWMMXT2_BUILTIN): Likewise.
> (IWMMXT2_BUILTIN2): Likewise.
> (FP_BUILTIN): Likewise.
> (CRC32_BUILTIN): Likewise.
> (CRYPTO_BUILTIN): Likewise.
> (iwmmx_mbuiltin): Likewise.
> (iwmmx2_mbuiltin): Likewise.
> (arm_init_iwmmxt_builtins): Likewise. Also, update for change to
> struct builtin_description.
>
OK - please watch out for any fallout in the next couple of days with
any auto-testers.
Ramana