From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 83760 invoked by alias); 17 May 2018 09:52:01 -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 83737 invoked by uid 89); 17 May 2018 09:52:00 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.9 required=5.0 tests=BAYES_00,KAM_LAZY_DOMAIN_SECURITY autolearn=no version=3.3.2 spammy=Hx-languages-length:1538 X-HELO: foss.arm.com Received: from foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Thu, 17 May 2018 09:51:59 +0000 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.72.51.249]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id 5E2951435; Thu, 17 May 2018 02:51:58 -0700 (PDT) Received: from [10.2.207.77] (e100706-lin.cambridge.arm.com [10.2.207.77]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 463CA3F24A; Thu, 17 May 2018 02:51:57 -0700 (PDT) Message-ID: <5AFD50BB.6030200@foss.arm.com> Date: Thu, 17 May 2018 09:56:00 -0000 From: Kyrill Tkachov User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:31.0) Gecko/20100101 Thunderbird/31.2.0 MIME-Version: 1.0 To: Tamar Christina , "gcc-patches@gcc.gnu.org" CC: nd , James Greenhalgh , Richard Earnshaw , Marcus Shawcroft Subject: Re: [PATCH][GCC][AArch64] Correct 3 way XOR instructions adding missing patterns. References: <20180430141233.GA28350@arm.com> In-Reply-To: <20180430141233.GA28350@arm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2018-05/txt/msg00786.txt.bz2 Hi Tamar, On 30/04/18 15:12, Tamar Christina wrote: > Hi All, > > This patch adds the missing neon intrinsics for all 128 bit vector Integer modes for the > three-way XOR and negate and xor instructions for Arm8.2-a to Armv8.4-a. > > Bootstrapped and regtested on aarch64-none-linux-gnue and no issues. > > Ok for master? And for backport to the GCC-8 branch? > This looks ok to me and appropriate for trunk but you'll need approval from a maintainer. Thanks, Kyrill > gcc/ > 2018-04-30 Tamar Christina > > * config/aarch64/aarch64-simd.md (aarch64_eor3qv8hi): Change to > eor3q4. > (aarch64_bcaxqv8hi): Change to bcaxq4. > * config/aarch64/aarch64-simd-builtins.def (veor3q_u8, veor3q_u32, > veor3q_u64, veor3q_s8, veor3q_s16, veor3q_s32, veor3q_s64, vbcaxq_u8, > vbcaxq_u32, vbcaxq_u64, vbcaxq_s8, vbcaxq_s16, vbcaxq_s32, > vbcaxq_s64): New. > * config/aarch64/arm_neon.h: Likewise. > * config/aarch64/iterators.md (VQ_I): New. > > gcc/testsuite/ > 2018-04-30 Tamar Christina > > * gcc.target/gcc.target/aarch64/sha3.h (veor3q_u8, veor3q_u32, > veor3q_u64, veor3q_s8, veor3q_s16, veor3q_s32, veor3q_s64, vbcaxq_u8, > vbcaxq_u32, vbcaxq_u64, vbcaxq_s8, vbcaxq_s16, vbcaxq_s32, > vbcaxq_s64): New. > * gcc.target/gcc.target/aarch64/sha3_1.c: Likewise. > * gcc.target/gcc.target/aarch64/sha3_1.c: Likewise. > * gcc.target/gcc.target/aarch64/sha3_1.c: Likewise. > > Thanks, > Tamar > > --