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

Vladimir Kozlov Vladimir.Kozlov at Sun.COM
Fri Mar 6 00:45:49 PST 2009


Looks good.

In x86_32.ad in loadB2L you need only 7 bits shift: SAR dst.hi, 7
and in loadS2L only 15 bits.

Vladimir

Tom Rodriguez wrote:
> Looks good to me.
> 
> tom
> 
> On Mar 3, 2009, at 1:10 PM, Christian Thalinger wrote:
> 
>> On Tue, 2009-03-03 at 19:08 +0100, Christian Thalinger wrote:
>>> Yeah, I will do that.  I will post a webrev later and open a new CR for
>>> the Address changes on SPARC.
>>
>> Here is the second version:
>>
>> http://cr.openjdk.java.net/~twisti/6797305/webrev.01/
>>
>> -- Christian
>>
> 



More information about the hotspot-compiler-dev mailing list