From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 92517 invoked by alias); 13 Nov 2015 11:24:50 -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 90732 invoked by uid 89); 13 Nov 2015 11:24:48 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-1.7 required=5.0 tests=AWL,BAYES_00,SPF_PASS autolearn=ham version=3.3.2 X-HELO: eu-smtp-delivery-143.mimecast.com Received: from eu-smtp-delivery-143.mimecast.com (HELO eu-smtp-delivery-143.mimecast.com) (146.101.78.143) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Fri, 13 Nov 2015 11:24:48 +0000 Received: from cam-owa1.Emea.Arm.com (fw-tnat.cambridge.arm.com [217.140.96.140]) by eu-smtp-1.mimecast.com with ESMTP id uk-mta-3-1UeTv_o4TvC6tOB693twdg-1; Fri, 13 Nov 2015 11:24:42 +0000 Received: from [10.2.206.200] ([10.1.2.79]) by cam-owa1.Emea.Arm.com with Microsoft SMTPSVC(6.0.3790.3959); Fri, 13 Nov 2015 11:24:41 +0000 Message-ID: <5645C879.7090406@arm.com> Date: Fri, 13 Nov 2015 11:24: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: Ramana Radhakrishnan , Christophe Lyon CC: Ramana Radhakrishnan , Jim Wilson , "gcc-patches@gcc.gnu.org" Subject: Re: [PATCH] [ARM] neon-testgen.ml typo References: <56337FD6.5040602@foss.arm.com> In-Reply-To: X-MC-Unique: 1UeTv_o4TvC6tOB693twdg-1 Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: quoted-printable X-IsSubscribed: yes X-SW-Source: 2015-11/txt/msg01664.txt.bz2 On 13/11/15 11:18, Ramana Radhakrishnan wrote: >> Hmm. I hadn't noticed that the crypto intrinsics tests where generated by >> neon-testgen.ml, I thought they were hand-written. >> The tests I added do not cover the crypto intrinsics, so I'm going >> to revert r230274 and restore all the tests generated by neon-testgen.ml >> until we have better coverage in advsimd-intrinsics. > From what I remember from a few days back I thought it was generally > ok to get rid of the lot as we had test coverage for everything else > in gcc.target/arm . > > Thus don't bother reverting. +1. I'll also add that you can now remove neon.ml from config/arm. And also, I think we can move the remaining hand-written tests from gcc.tar= get/arm/neon/ into gcc.target/arm/ and remove the neon/ directory altogether. Kyrill > > Ramana > >> Sorry for the oversight. >> >> Christophe. >> >> >>> Christophe. >>> >>>>> regards >>>>> Ramana >>>>> >>>>> >>>>> >>>>>> Christophe. >>>>>> >>>>>>> regards >>>>>>> Ramana