From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from foss.arm.com (foss.arm.com [217.140.110.172]) by sourceware.org (Postfix) with ESMTP id 939F33858409 for ; Thu, 14 Oct 2021 13:44:44 +0000 (GMT) DMARC-Filter: OpenDMARC Filter v1.4.1 sourceware.org 939F33858409 Received: from usa-sjc-imap-foss1.foss.arm.com (unknown [10.121.207.14]) by usa-sjc-mx-foss1.foss.arm.com (Postfix) with ESMTP id D847CD6E; Thu, 14 Oct 2021 06:44:43 -0700 (PDT) Received: from [10.1.37.147] (E121495.Arm.com [10.1.37.147]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 5F9F23F70D; Thu, 14 Oct 2021 06:44:43 -0700 (PDT) Message-ID: <4d6c7fe3-e3a0-77f7-cb68-2d928d2b554b@arm.com> Date: Thu, 14 Oct 2021 14:44:51 +0100 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (Windows NT 10.0; Win64; x64; rv:91.0) Gecko/20100101 Thunderbird/91.2.0 Subject: Re: [PATCH 2/3][vect] Consider outside costs earlier for epilogue loops Content-Language: en-US To: "gcc-patches@gcc.gnu.org" Cc: Richard Sandiford , Richard Biener References: <4a2e6dde-cc5c-97fe-7a43-bd59d542c2ce@arm.com> <4b403865-bb56-29a4-56d0-b18536925db6@arm.com> From: "Andre Vieira (lists)" In-Reply-To: <4b403865-bb56-29a4-56d0-b18536925db6@arm.com> Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: 8bit X-Spam-Status: No, score=-5.1 required=5.0 tests=BAYES_00, BODY_8BITS, KAM_DMARC_STATUS, NICE_REPLY_A, SPF_HELO_NONE, SPF_PASS, TXREP autolearn=ham autolearn_force=no version=3.4.4 X-Spam-Checker-Version: SpamAssassin 3.4.4 (2020-01-24) on server2.sourceware.org X-BeenThere: gcc-patches@gcc.gnu.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Gcc-patches mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Thu, 14 Oct 2021 13:44:46 -0000 Hi, I completely forgot I still had this patch out as well, I grouped it together with the unrolling because it was what motivated the change, but it is actually wider applicable and can be reviewed separately. On 17/09/2021 16:32, Andre Vieira (lists) via Gcc-patches wrote: > Hi, > > This patch changes the order in which we check outside and inside > costs for epilogue loops, this is to ensure that a predicated epilogue > is more likely to be picked over an unpredicated one, since it saves > having to enter a scalar epilogue loop. > > gcc/ChangeLog: > >         * tree-vect-loop.c (vect_better_loop_vinfo_p): Change how > epilogue loop costs are compared.