Request for review (L): 6797305: Add LoadUB and LoadUI opcode class

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Fri Mar 6 14:05:05 PST 2009


I am fine also with this.

Thanks,
Vladimir

Tom Rodriguez wrote:
> I'm fine with you committing it, though could you check that the changes 
> to cisc_spill_match don't change the result of any other cisc spill 
> decisions on 32 bit intel?
> 
> tom
> 
> On Mar 6, 2009, at 2:04 PM, Christian Thalinger wrote:
> 
>> On Fri, 2009-03-06 at 13:34 -0800, Tom Rodriguez wrote:
>>> This will verify that the ideal type of the register and the load
>>> operation are in agreement.
>>
>> Yes, that fixes it.  Thanks a lot!  Should I integrate these changes
>> into my patch and commit it or should we do another review round?
>>
>> -- Christian
>>
> 



More information about the hotspot-compiler-dev mailing list