[prev in list] [next in list] [prev in thread] [next in thread] 

List:       openjdk-hotspot-compiler-dev
Subject:    Request for review (L): 6797305: Add LoadUB and LoadUI opcode	class
From:       Thomas.Rodriguez () Sun ! COM (Tom Rodriguez)
Date:       2009-02-27 22:58:31
Message-ID: 323398B3-5BE1-415D-8E4B-0073BD434683 () Sun ! COM
[Download RAW message or body]


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
>



[prev in list] [next in list] [prev in thread] [next in thread]