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] Variable Expansion in unroller + Register Pressure Utility


> Now I see that Revital added function 
> estimate_register_pressure with a register as parameter which I don't 
> understand.  It just returns difference between number of available hard 

> registers in class (which the pseudo-register belongs to for register 
> pressure estimation purpose) and maximal register pressure in some 
> blocks.  Even if we are trying to create a pseudo-register in the basic 
> blocks, the returned value says nothing because the new pseudo-register 
> could be used in some point where the register pressure does not reach 
> maximal value.  So with my point of view it is not accurate evaluation 
> to decide create or not the new pseudo-register.
> 
> Vlad

This is why among others it is highly conservative approach. 
I can change it to use per-block register pressure check
if you think it is more accurate.

Thanks,
Revital 


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