This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [RFA] The Integrated Register Allocator


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.



Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]