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