From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 84394 invoked by alias); 1 Aug 2018 10:20:57 -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 84380 invoked by uid 89); 1 Aug 2018 10:20:55 -0000 Authentication-Results: sourceware.org; auth=none X-Spam-SWARE-Status: No, score=-1.9 required=5.0 tests=AWL,BAYES_00,RCVD_IN_DNSWL_NONE,SPF_HELO_PASS,SPF_PASS autolearn=ham version=3.3.2 spammy= X-HELO: EUR04-DB3-obe.outbound.protection.outlook.com Received: from mail-eopbgr60079.outbound.protection.outlook.com (HELO EUR04-DB3-obe.outbound.protection.outlook.com) (40.107.6.79) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Wed, 01 Aug 2018 10:20:53 +0000 DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=armh.onmicrosoft.com; s=selector1-arm-com; h=From:Date:Subject:Message-ID:Content-Type:MIME-Version:X-MS-Exchange-SenderADCheck; bh=2KRUFMyYQmfHpg7FVeSash3m6vaZJawV0x3RjHsMiI4=; b=Fc/GUjfRHcumeyU5skevxCpDKKcPSxsRq2iXc83nTJIOejFvPVlCiSvUQVNw5slsN0l1Bdljz6bWBf26/93iP6j74mjEnAKgAsD8TjAkI4aZeF3XaWyIIwJTrC80xSUOzqpbqaTwbE7kjnNIkgchmmo/rehupt+lZu+1HD+nXv8= Authentication-Results: spf=none (sender IP is ) smtp.mailfrom=Sudi.Das@arm.com; Received: from [10.2.206.246] (217.140.106.6) by DB6PR0801MB1720.eurprd08.prod.outlook.com (2603:10a6:4:3a::19) with Microsoft SMTP Server (version=TLS1_2, cipher=TLS_ECDHE_RSA_WITH_AES_256_GCM_SHA384) id 15.20.995.17; Wed, 1 Aug 2018 10:20:47 +0000 Subject: Re: [GCC][PATCH][Aarch64] Stop redundant zero-extension after UMOV when in DI mode To: Sam Tebbs , "gcc-patches@gcc.gnu.org" Cc: James Greenhalgh , Marcus Shawcroft , nd , Richard Earnshaw References: <953dbdd2-e20c-4587-3e0d-ad1a65fc93c6@arm.com> <85b58ddb-3da2-67c6-1514-e308201191d3@arm.com> <74da7cb6-485b-3ce4-7901-d10cb6f1ed95@arm.com> <9e7ccf5b-55b9-543d-1f9e-f9ab36e93376@arm.com> <20180731221625.GH1826@arm.com> <6be85f18-5f4c-0312-cc0a-bc1b69e9a183@arm.com> From: Sudakshina Das Message-ID: <731e9538-0d3e-c7df-0ae8-5c6d63c04dc6@arm.com> Date: Wed, 01 Aug 2018 10:20:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.6.0 MIME-Version: 1.0 In-Reply-To: <6be85f18-5f4c-0312-cc0a-bc1b69e9a183@arm.com> Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 8bit Return-Path: sudi.das@arm.com Received-SPF: None (protection.outlook.com: arm.com does not designate permitted sender hosts) X-IsSubscribed: yes X-SW-Source: 2018-08/txt/msg00029.txt.bz2 Hi Sam On 01/08/18 10:12, Sam Tebbs wrote: > > > On 07/31/2018 11:16 PM, James Greenhalgh wrote: >> On Thu, Jul 26, 2018 at 11:52:15AM -0500, Sam Tebbs wrote: >> >> >> >>>> Thanks for making the changes and adding more test cases. I do however >>>> see that you are only covering 2 out of 4 new >>>> *aarch64_get_lane_zero_extenddi<> patterns. The >>>> *aarch64_get_lane_zero_extendsi<> were already existing. I don't mind >>>> those tests. I would just ask you to add the other two new patterns >>>> as well. Also since the different versions of the instruction generate >>>> same instructions (like foo_16qi and foo_8qi both give out the same >>>> instruction), I would suggest using a -fdump-rtl-final (or any relevant >>>> rtl dump) with the dg-options and using a scan-rtl-dump to scan the >>>> pattern name. Something like: >>>> /* { dg-do compile } */ >>>> /* { dg-options "-O3 -fdump-rtl-final" } */ >>>> ... >>>> ... >>>> /* { dg-final { scan-rtl-dump "aarch64_get_lane_zero_extenddiv16qi" >>>> "final" } } */ >>>> >>>> Thanks >>>> Sudi >>> Hi Sudi, >>> >>> Thanks again. Here's an update that adds 4 more tests, so all 8 patterns >>> generated are now tested for! >> This is OK for trunk, thanks for the patch (and thanks Sudi for the >> review!) >> >> Thanks, >> James > > Thank you James! I'd appreciate it if someone could commit it as I don't > have commit rights yet. > I have committed this on your behalf as r263200. Thanks Sudi > Sam > >> >>> Below is the updated changelog >>> >>> gcc/ >>> 2018-07-26  Sam Tebbs  >>> >>>           * config/aarch64/aarch64-simd.md >>>           (*aarch64_get_lane_zero_extendsi): >>>           Rename to... >>> (*aarch64_get_lane_zero_extend): ... This. >>>           Use GPI iterator instead of SI mode. >>> >>> gcc/testsuite >>> 2018-07-26  Sam Tebbs  >>> >>>           * gcc.target/aarch64/extract_zero_extend.c: New file >>> >