Request for reviews (XS): 6879902: CTW failure jdk6_18/hotspot/src/cpu/sparc/vm/assembler_sparc.hpp:845

[Christian Thalinger]

On Fri, 2009-10-02 at 12:32 +0200, Christian Thalinger wrote:
> Just for the record... I have already prepared another webrev but did
> had time yet to post it:
> 
> http://cr.openjdk.java.net/~twisti/6879902/webrev.02/
> 
> and Tom had a look at it:
> 
> I looked at this again and I don't think that stf should be using  
> arg_slot.  c2i only uses arg_slot/next_arg_slot for loads which is  
> correct since we should be moving it from interpreter layout to  
> compiled layout.  Using it for stores is wrong.  After Vladimir's  
> comment I thought that other compiled code was already using arg_slot  
> for stores so it wasn't any worse but now I can see that it's not  
> true.  I think the original fix of just correcting the stf code was  
> the way to go.  It can use RegisterOrConstant with ensure_rs2.  We can  
> keep the other changes or not depending on how you feel about them.   
> They generate better code for 64-bit and deals with the simm13 issue  
> in general which I like.  If you keep them I'd suggest using  
> ensure_rs2 in you arg_slot routines instead of hand coding that.  I  
> wouldn't mind renaming ensure_rs2 since it's such a useless name.

Here is an updated webrev:

http://cr.openjdk.java.net/~twisti/6879902/webrev.03/

-- Christian