From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (qmail 126185 invoked by alias); 4 Sep 2017 16:20:34 -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 126176 invoked by uid 89); 4 Sep 2017 16:20:34 -0000 Authentication-Results: sourceware.org; auth=none X-Virus-Found: No X-Spam-SWARE-Status: No, score=-0.9 required=5.0 tests=BAYES_00,KAM_LAZY_DOMAIN_SECURITY,RP_MATCHES_RCVD autolearn=no version=3.3.2 spammy= X-HELO: foss.arm.com Received: from usa-sjc-mx-foss1.foss.arm.com (HELO foss.arm.com) (217.140.101.70) by sourceware.org (qpsmtpd/0.93/v0.84-503-g423c35a) with ESMTP; Mon, 04 Sep 2017 16:20:29 +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 F1E5F80D; Mon, 4 Sep 2017 09:20:27 -0700 (PDT) Received: from [10.2.207.77] (e100706-lin.cambridge.arm.com [10.2.207.77]) by usa-sjc-imap-foss1.foss.arm.com (Postfix) with ESMTPSA id E4B3F3F3E1; Mon, 4 Sep 2017 09:20:26 -0700 (PDT) Message-ID: <59AD7D49.7010505@foss.arm.com> Date: Mon, 04 Sep 2017 16:20: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: Bernd Edlinger , "gcc-patches@gcc.gnu.org" CC: Ramana Radhakrishnan , Richard Earnshaw , Wilco Dijkstra Subject: Re: [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308) References: In-Reply-To: Content-Type: text/plain; charset=utf-8; format=flowed Content-Transfer-Encoding: 7bit X-SW-Source: 2017-09/txt/msg00195.txt.bz2 Hi Bernd, On 29/04/17 18:52, Bernd Edlinger wrote: > Ping... > > I attached the latest version of my patch. > > > Thanks > Bernd. > > On 12/18/16 14:14, Bernd Edlinger wrote: >> Hi, >> >> this splits the *arm_negdi2, *arm_cmpdi_insn and *arm_cmpdi_unsigned >> also at split1 except for TARGET_NEON and TARGET_IWMMXT. >> >> In the new test case the stack is reduced to about 270 bytes, except >> for neon and iwmmxt, where this does not change anything. >> >> This patch depends on [1] and [2] before it can be applied. >> >> Bootstrapped and reg-tested on arm-linux-gnueabihf. >> Is it OK for trunk? >> >> >> Thanks >> Bernd. >> >> >> >> [1] https://gcc.gnu.org/ml/gcc-patches/2016-11/msg02796.html >> [2] https://gcc.gnu.org/ml/gcc-patches/2016-12/msg01562.html 2016-12-18 Bernd Edlinger PR target/77308 * config/arm/arm.md (*arm_negdi2, *arm_cmpdi_insn, *arm_cmpdi_unsigned): Split early except for TARGET_NEON and TARGET_IWMMXT. You're changing negdi2_insn rather than *arm_negdi2. Ok with the fixed ChangeLog once the prerequisite is committed. Thanks, Kyrill