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: [PATCH] Fix PR middle-end/28690, indexed load/store performance + reload bug


Hi,

On Mon, 11 Dec 2006, Peter Bergner wrote:

> In this case, the first "find_reg (allocno_order[i], 0, 0, 0, 0)" fails 
> to find a register, so we're forced to find a register in an alternate 
> class. For pseudo 238, reg_alternate_class() returns ALL_REGS,

Okay, that makes sense I guess (the preferred class is all used already so 
it has to defer to the alternate class).  ALL_REGS isn't shown in the 
dumps, and I confused it with NO_REGS (which actually is shown as "or 
none"), hence assumed wrongly that it got NO_REGS as alternate class.  
With ALL_REGS the choice of 65 makes sense and is expected and the 
predicate needs to be augmented by a check for reload_in_progress.  The 
choice of ALL_REGS seems also to be correct to me.


Ciao,
Michael.


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