From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 130642 invoked by alias); 6 Jan 2017 10:53:38 -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 130630 invoked by uid 89); 6 Jan 2017 10:53:37 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-5.1 required=5.0 tests=BAYES_00,RP_MATCHES_RCVD,SPF_PASS autolearn=ham version=3.3.2 spammy= 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; Fri, 06 Jan 2017 10:53:36 +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 E880E1610; Fri, 6 Jan 2017 02:53:33 -0800 (PST) Received: from [10.2.206.251] (e107157-lin.cambridge.arm.com [10.2.206.251]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id 573E53F220; Fri, 6 Jan 2017 02:53:33 -0800 (PST) Subject: Re: [PATCH] PR78255: Make postreload aware of NO_FUNCTION_CSE To: GCC Patches References: <584AB9AA.6030800@arm.com> <334ff580-3e7d-22fb-83da-da18acd84244@redhat.com> <584ACF02.9070101@arm.com> <584AD8E9.3000406@arm.com> Cc: Jakub Jelinek , Richard Biener From: "Andre Vieira (lists)" Message-ID: <586F772B.50209@arm.com> Date: Fri, 06 Jan 2017 10:53:00 -0000 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:38.0) Gecko/20100101 Thunderbird/38.2.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Transfer-Encoding: 7bit X-IsSubscribed: yes X-SW-Source: 2017-01/txt/msg00370.txt.bz2 On 09/12/16 16:31, Bernd Schmidt wrote: > On 12/09/2016 05:16 PM, Andre Vieira (lists) wrote: > >> Regardless, 'reload_cse_simplify' would never perform the opposite >> transformation. It checks whether it can replace anything within the >> first argument INSN, with the second argument TESTREG. As the name >> implies this will always be a register. I double checked, the function >> is only called in 'reload_cse_regs' and 'testreg' is created using >> 'gen_rtx_REG'. > > Ok, let's go ahead with it. > > > Bernd > Hello, Is it OK to backport this (including the testcase fix) to gcc-6-branch? Patches apply cleanly and full bootstrap and regression tests for aarch64- and arm-none-linux-gnueabihf. Regression tested for arm-none-eabi. Cheers, Andre