From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 17436 invoked by alias); 5 Nov 2014 23:31:39 -0000 Mailing-List: contact gcc-patches-help@gcc.gnu.org; run by ezmlm Precedence: bulk List-Id: List-Archive: List-Post: List-Help: Sender: gcc-patches-owner@gcc.gnu.org Received: (qmail 17426 invoked by uid 89); 5 Nov 2014 23:31:39 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.3 required=5.0 tests=AWL,BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 X-HELO: cam-smtp0.cambridge.arm.com Received: from fw-tnat.cambridge.arm.com (HELO cam-smtp0.cambridge.arm.com) (217.140.96.21) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Wed, 05 Nov 2014 23:31:29 +0000 Received: from arm.com (e106375-lin.cambridge.arm.com [10.1.203.34]) by cam-smtp0.cambridge.arm.com (8.13.8/8.13.8) with ESMTP id sA5NVO6v010395; Wed, 5 Nov 2014 23:31:24 GMT Date: Wed, 05 Nov 2014 23:31:00 -0000 From: James Greenhalgh To: Marc Glisse Cc: Tejas Belagod , "gcc-patches@gcc.gnu.org" Subject: Re: [AArch64, Patch] Restructure arm_neon.h vector types's implementation(Take 2). Message-ID: <20141105233124.GA29096@arm.com> References: <542BBAB9.9080703@arm.com> <5459DFC9.3010700@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: User-Agent: Mutt/1.5.21 (2010-09-15) X-IsSubscribed: yes X-SW-Source: 2014-11/txt/msg00384.txt.bz2 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 > >>> > >>> * 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