patch to enable LRA for ppc

Vladimir Makarov vmakarov@redhat.com
Mon Oct 21 07:33:00 GMT 2013


On 13-10-18 11:26 AM, David Edelsohn wrote:
> On Thu, Oct 3, 2013 at 5:02 PM, Vladimir Makarov <vmakarov@redhat.com> wrote:
>> The following patch permits today trunk to use LRA for ppc by default.
>> To switch it off -mno-lra can be used.
>>
>> The patch was bootstrapped on ppc64.  GCC testsuite does not have
>> regressions too (in comparison with reload).  The change in rs6000.md is
>> for fix LRA failure on a recently added ppc test.
> Vlad,
>
> I have not forgotten this patch. We are trying to figure out the right
> timeframe to make this change. The patch does affect performance --
> both positively and negatively; most are in the noise but not all. And
> there still are some SPEC benchmarks that fail to build with the
> patch, at least in Mike's tests. And Mike is implementing some patches
> to utilize reload to improve use of VSX registers, which would need to
> be mirrored in LRA for the equivalent functionality.
Thanks for informing me, David.

I am ready to work on any LRA ppc issues when it will be in the trunk.  
It would be easier for me to work on LRA ppc if the patch is committed 
to the trunk and of course LRA is used as non-default local RA.

I don't know what Mike is doing on reload to use VSX registers.  I guess 
it is usage of  VSX regs as spilled locations for GENERAL regs instead 
of memory.  If it is so, it is 2 day work to add this functionality in 
LRA (as it already has analogous functionality for Intel processors and 
that gave a nice SPECFP2000 improvement for them) and probably more work 
on resolving issues especially as I have no power8.





More information about the Gcc-patches mailing list