This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)
- From: Kyrill Tkachov <kyrylo dot tkachov at foss dot arm dot com>
- To: Bernd Edlinger <bernd dot edlinger at hotmail dot de>, "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>
- Cc: Ramana Radhakrishnan <ramana dot radhakrishnan at arm dot com>, Richard Earnshaw <richard dot earnshaw at arm dot com>, Wilco Dijkstra <wilco dot dijkstra at arm dot com>
- Date: Mon, 04 Sep 2017 17:20:25 +0100
- Subject: Re: [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)
- Authentication-results: sourceware.org; auth=none
- References: <AM4PR0701MB216257873FAD30D4E8975FF0E49E0@AM4PR0701MB2162.eurprd07.prod.outlook.com> <AM4PR0701MB216270761128D367F882BEC2E4120@AM4PR0701MB2162.eurprd07.prod.outlook.com>
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<bernd.edlinger@hotmail.de>
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