[PATCH, ARM] Further improve stack usage in sha512, part 2 (PR 77308)
Kyrill Tkachov
kyrylo.tkachov@foss.arm.com
Mon Sep 4 16:20:00 GMT 2017
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
More information about the Gcc-patches
mailing list