[RFA] The Integrated Register Allocator

Vladimir Makarov vmakarov@redhat.com
Fri Apr 4 19:47:00 GMT 2008


Vladimir Makarov wrote:
> Pat Haugen wrote:
>>> Even for targets with many hard registers (and that is a surprise for
>>> me) like ppc64 and itanium, there is a still a small improvement on
>>> SPECINT2000 (about 1%).
>>>
>>>     
>>
>> I tried your patch on ppc64 (a Power6 box). For the most part results 
>> were
>> close, but 187.facerec degraded about 24% with -fira.  It looks like the
>> hot procedure, graphRoutines.f90:LocalMove(), ended up with a lot more
>> spill code.
>>   
> Thanks for testing.
>
> I know about the problem, it was occurred as the result of latest 
> patches.  IRA by mistake assigns FLOAT_REGS instead of GENERAL_REGS 
> for some pseudos.  The wrong code results in code size degradation too 
> on ppc64 and itanium.
>
> I have the patch for fixing this.  I'll commit it to the branch today.

I've just committed a patch fixing the problem.  Sorry for 
inconvenience.  I've should done it earlier.



More information about the Gcc-patches mailing list