Request for review (XL): 6822110: Add AddressLiteral class on SPARC

Christian Thalinger Christian.Thalinger at Sun.COM
Thu Apr 16 08:34:57 PDT 2009


On Wed, 2009-04-01 at 21:40 +0200, Christian Thalinger wrote:
> > assembler_sparc.hpp:
> > 
> > why does load_contents take 2 registers?  All uses pass the same thing  
> > for both.  It should just assume the dest is the temp.  Actually I  
> > think that's true of all the new ones that take both a temp and a dest.
> 
> That sounds reasonable.  I will change that.

Finally I found the bugs I had in my changes and here is the third, and
hopefully final, webrev:

http://cr.openjdk.java.net/~twisti/6822110/webrev.02/

I tested the changes with JPRT, DaCapo and SPECjvm2008.

-- Christian




More information about the hotspot-compiler-dev mailing list