This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).
- From: James Greenhalgh <james dot greenhalgh at arm dot com>
- To: Marc Glisse <marc dot glisse at inria dot fr>
- Cc: Tejas Belagod <Tejas dot Belagod at arm dot com>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Date: Wed, 5 Nov 2014 23:31:24 +0000
- Subject: Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2).
- Authentication-results: sourceware.org; auth=none
- References: <542BBAB9 dot 9080703 at arm dot com> <CAFqB+Px7y_qBr2uOKmrDie7z2kpFUR=waNk=N4j935cRfLgucA at mail dot gmail dot com> <5459DFC9 dot 3010700 at arm dot com> <alpine dot DEB dot 2 dot 11 dot 1411052236160 dot 1572 at laptop-mg dot saclay dot inria dot fr>
On Wed, Nov 05, 2014 at 09:50:52PM +0000, Marc Glisse wrote:
> On Wed, 5 Nov 2014, Tejas Belagod wrote:
>
> >>> 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.
>
> Thanks. Do you know if anyone is planning to "port" this patch to the arm
> target (which IIRC has the same issue)? No pressure, this is just so I
> know if I should ping the alternate __float128 patch or wait.
Hi Marc,
Yes, you're right, the arm target also has this issue. I have a port of
Tejas' patch based on top of some other refactoring of the arm
target's builtin infrastructure I've been working on - I'm hoping to get
that all rebased and proposed upstream in the next couple of weeks.
Thanks,
James