Re: [PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)

[Kyrill Tkachov]

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.edlin...@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