From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 78915 invoked by alias); 16 Jun 2015 13:24:13 -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 78899 invoked by uid 89); 16 Jun 2015 13:24:13 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-2.5 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_LOW,SPF_PASS autolearn=ham version=3.3.2 X-HELO: mail-qc0-f180.google.com Received: from mail-qc0-f180.google.com (HELO mail-qc0-f180.google.com) (209.85.216.180) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with (AES128-GCM-SHA256 encrypted) ESMTPS; Tue, 16 Jun 2015 13:24:10 +0000 Received: by qcbfz6 with SMTP id fz6so4011506qcb.0 for ; Tue, 16 Jun 2015 06:24:08 -0700 (PDT) X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20130820; h=x-gm-message-state:mime-version:in-reply-to:references:date :message-id:subject:from:to:cc:content-type; bh=toWo/8Lo0oSSz4w7FzKCTa2EjoL80wd2lVcujzeTgfo=; b=cfoL0biD+NDWJnUI+nCE0KHvAZjGP9ko27ZsTMEE/S4Uzb+2aG+uZbH/sgB0n2Y3YM Co9/qLZAu1VGtCEB8YuDVkDBtVjoBOgvUSVEYpmadmhHvp6mTSEriRQTtHuK8To+KOHD 6GmVpQYbfzRF0oKRlEsUpYPoB2usAEE3qHcuzHiFMAwi3KLKXMnKd6ORSR+IinxQjbaD 0XTECtA10kXXxbLAlbvUvY3J7MBbKhPMk6hDDYigul+8s3LM7g06Lbct/FAWHja95eJ3 qgpvNS0JROKgaFzjPwR3c6PF6fqjkaoU3MN0XknY63bhpzuSKp3YdW1XgRN4yLdZY1Ha 9WsA== X-Gm-Message-State: ALoCoQkdejvIzBqwCbn2BUKzepToAMqUx6WsLPiFUVpmySEeQ63FYCNjywH3TLaX4OlhinYT8toq MIME-Version: 1.0 X-Received: by 10.55.26.167 with SMTP id l39mr978978qkh.4.1434461048372; Tue, 16 Jun 2015 06:24:08 -0700 (PDT) Received: by 10.140.102.164 with HTTP; Tue, 16 Jun 2015 06:24:08 -0700 (PDT) In-Reply-To: <20150616100849.GA31808@arm.com> References: <1432757747-4891-1-git-send-email-christophe.lyon@linaro.org> <20150616100849.GA31808@arm.com> Date: Tue, 16 Jun 2015 13:26:00 -0000 Message-ID: Subject: Re: [Patch ARM-AArch64/testsuite Neon intrinsics 00/20] Executable tests From: Christophe Lyon To: James Greenhalgh Cc: "gcc-patches@gcc.gnu.org" Content-Type: text/plain; charset=UTF-8 X-IsSubscribed: yes X-SW-Source: 2015-06/txt/msg01122.txt.bz2 On 16 June 2015 at 12:08, James Greenhalgh wrote: > On Mon, Jun 15, 2015 at 11:11:16PM +0100, Christophe Lyon wrote: >> Ping? >> >> >> On 27 May 2015 at 22:15, Christophe Lyon wrote: >> > This patch series is a follow-up to the tests I already contributed, >> > converted from my original testsuite. >> > >> > This series consists in 20 new patches, which can be committed >> > independently. For vrecpe, I added the setting of the "Flush-to-Zero" >> > FP flag, to force AArch64 to behave the same as ARM by default. >> > >> > This is the final batch, except for the vget_lane tests which I will >> > submit later. This should cover the subset of AdvSIMD intrinsics >> > common to ARMv7 and AArch64. >> > >> > Tested with qemu on arm*linux, aarch64-linux. >> > >> > 2015-05-27 Christophe Lyon >> > >> > * gcc.target/aarch64/advsimd-intrinsics/arm-neon-ref.h >> > (_ARM_FPSCR): Add FZ field. >> > (clean_results): Force FZ=1 on AArch64. >> > * gcc.target/aarch64/advsimd-intrinsics/vrecpe.c: New file. >> > * gcc.target/aarch64/advsimd-intrinsics/vrecps.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vreinterpret.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrev.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrshl.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrshr_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrshrn_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrsqrte.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrsqrts.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vrsra_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vset_lane.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vshl_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vshll_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vshr_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vshrn_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vsra_n.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vst1_lane.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vstX_lane.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vtbX.c: Likewise. >> > * gcc.target/aarch64/advsimd-intrinsics/vtst.c: Likewise. >> > > > This patch set is OK. > > As with the last patch set, please do a quick run through of each patch > before committing and ensure that the trailing '\' characters line up, > and look for any fall-out (particularly on aarch64_be) after these are in. > I've just re-run the tests on aarch64_be, and the vtbX one is failing when testing vtlb3. It does pass on armeb, so I expect the test to be correctly written to be endianness independant. As a reminder, there are still several other tests failing on aarch64_be: vldX_lane vtrn vzip vuzp we have PR63652 and 63653 for these. > Thanks, > James >