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

John Rose John.Rose at Sun.COM
Tue Mar 10 10:26:21 PDT 2009


On Mar 10, 2009, at 5:08 AM, Christian Thalinger wrote:

> For the load shortening changes I'm currently doing it would be useful
> to have these.  Can I add them under a new CR?

There is something like that already in assembler_sparc.hpp:
      inline void lduw( const Address& a, Register d, int offset = 0 );

I thought you were talking about making Address easier to use...?

If you just want to add a +4 offset, you can use the optional offset  
argument above.

If you want to fold the offset into the Address itself, use  
Address::plus_disp (which is new).

-- John



More information about the hotspot-compiler-dev mailing list