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