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

Tom Rodriguez Thomas.Rodriguez at Sun.COM
Fri Feb 27 14:58:31 PST 2009


On Feb 27, 2009, at 1:34 PM, Christian Thalinger wrote:

> On Fri, 2009-02-27 at 10:06 -0800, Tom Rodriguez wrote:
>> Basically you have to distinguish the two modes by hand.  Check out
>> the loadN instruct definition.
>
> Okay, I will have a look at it.
>
>> Does $mem$$base$$Register not work?
>
> No, because it generates:
>
> opnd_array(1)->base(ra_,this,idx1)->as_Register(ra_,this,idx1)
>
> and base() already returns an int.

I've added some more parsing lookahead to deal with this case properly  
and updated the address webrev to include it.  $mem$$base$$Register  
and $mem$$index$$Register should now emit properly.

tom

>
>
> -- Christian
>




More information about the hotspot-compiler-dev mailing list