Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).

[Tejas Belagod <tejas dot belagod at arm dot com>]

On 03/11/14 16:49, Marcus Shawcroft wrote:
> On 1 October 2014 09:26, Tejas Belagod <tejas.belagod@arm.com> wrote:
> > Hi,
> >
> > Returning to this old thread,
> >
> > https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02285.html
> >
> > here is a patch after a few rounds of review comments, specifically:
> >
> > https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02248.html
> > https://gcc.gnu.org/ml/gcc-patches/2014-06/msg02285.html
> > https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00566.html
> > https://gcc.gnu.org/ml/gcc-patches/2014-07/msg00324.html
> >
> > Sorry for the delay in respinning this patch.
> >
> > Tested on aarch64-none-elf. OK for trunk?
> >
> > Thanks,
> > Tejas.
> >
> > Changelog:
> >
> > 2014-10-01  Tejas Belagod  <tejas.belagod@arm.com>
> >
> >          * config/aarch64/aarch64-builtins.c
> >          (aarch64_build_scalar_type): Remove.
> >          (aarch64_scalar_builtin_types, aarch64_simd_type,
> >          aarch64_simd_type, aarch64_mangle_builtin_scalar_type,
> >          aarch64_mangle_builtin_vector_type,
> >          aarch64_mangle_builtin_type, aarch64_simd_builtin_std_type,
> >          aarch64_lookup_simd_builtin_type, aarch64_simd_builtin_type,
> >          aarch64_init_simd_builtin_types,
> >          aarch64_init_simd_builtin_scalar_types): New.
> >          (aarch64_init_simd_builtins): Refactor.
> >          (aarch64_init_crc32_builtins): Fixup with qualifier.
> >          * config/aarch64/aarch64-protos.h
> >          (aarch64_mangle_builtin_type): Export.
> >          * config/aarch64/aarch64-simd-builtin-types.def: New.
> >          * config/aarch64/aarch64.c (aarch64_simd_mangle_map): Remove.
> >          (aarch64_mangle_type): Refactor.
> >          * config/aarch64/arm_neon.h: Declare vector types based on
> >          internal types.
> >          * config/aarch64/t-aarch64: Update dependency.
>
> OK /Marcus

Thanks. Rebased, retested on aarch64-none-elf and committed as r217114.

Tejas.