From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 98912 invoked by alias); 3 Aug 2015 11:25:03 -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 98902 invoked by uid 89); 3 Aug 2015 11:25:02 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.4 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.140) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES256-SHA encrypted) ESMTPS; Mon, 03 Aug 2015 11:25:01 +0000 Received: from arm.com (e106375-lin.cambridge.arm.com [10.2.207.23]) by cam-smtp0.cambridge.arm.com (8.13.8/8.13.8) with ESMTP id t73BOwcT014070; Mon, 3 Aug 2015 12:24:58 +0100 Date: Mon, 03 Aug 2015 11:25:00 -0000 From: James Greenhalgh To: Kyrill Tkachov Cc: GCC Patches , Marcus Shawcroft , Richard Earnshaw Subject: Re: [PATCH][AArch64][11/14] Re-layout SIMD builtin types on builtin expansion Message-ID: <20150803112458.GC10971@arm.com> References: <55A7CBE7.4010204@arm.com> <55AE7A7B.9030707@arm.com> <20150722091137.GA14893@arm.com> <55B1F98A.3070307@arm.com> MIME-Version: 1.0 Content-Type: text/plain; charset=us-ascii Content-Disposition: inline In-Reply-To: <55B1F98A.3070307@arm.com> User-Agent: Mutt/1.5.21 (2010-09-15) X-IsSubscribed: yes X-SW-Source: 2015-08/txt/msg00048.txt.bz2 On Fri, Jul 24, 2015 at 09:38:34AM +0100, Kyrill Tkachov wrote: > Thanks, here's an updated version. > > 2015-07-24 Kyrylo Tkachov > > * config/aarch64/aarch64.c (aarch64_option_valid_attribute_p): > Initialize simd builtins if TARGET_SIMD. > * config/aarch64/aarch64-builtins.c (aarch64_init_simd_builtins): > Make sure that the builtins are initialized only once no matter how > many times the function is called. > (aarch64_init_builtins): Unconditionally initialize crc builtins. > (aarch64_relayout_simd_param): New function. > (aarch64_simd_expand_args): Use above during argument expansion. > * config/aarch64/aarch64-c.c (aarch64_pragma_target_parse): Initialize > simd builtins if TARGET_SIMD. > * config/aarch64/aarch64-protos.h (aarch64_init_simd_builtins): New > prototype. > (aarch64_relayout_simd_types): Likewise. > > 2015-07-24 Kyrylo Tkachov > > * gcc.target/aarch64/target_attr_crypto_ice_1.c: New test. > OK with a minor fix. > diff --git a/gcc/config/aarch64/aarch64-protos.h b/gcc/config/aarch64/aarch64-protos.h > index 6844c90..99fd80e 100644 > --- a/gcc/config/aarch64/aarch64-protos.h > +++ b/gcc/config/aarch64/aarch64-protos.h > @@ -255,6 +255,7 @@ bool aarch64_float_const_zero_rtx_p (rtx); > bool aarch64_function_arg_regno_p (unsigned); > bool aarch64_gen_movmemqi (rtx *); > bool aarch64_gimple_fold_builtin (gimple_stmt_iterator *); > +void aarch64_init_simd_builtins (void); > bool aarch64_is_extend_from_extract (machine_mode, rtx, rtx); > bool aarch64_is_long_call_p (rtx); > bool aarch64_label_mentioned_p (rtx); These should be first ordered by return type, then alphabetical order. > @@ -325,6 +326,7 @@ void aarch64_print_operand (FILE *, rtx, char); > void aarch64_print_operand_address (FILE *, rtx); > void aarch64_emit_call_insn (rtx); > void aarch64_register_pragmas (void); > +void aarch64_relayout_simd_types (void); > void aarch64_reset_previous_fndecl (void); > > /* Initialize builtins for SIMD intrinsics. */ Thanks, James