Request for reviews (XL): 6829193: JSR 292 needs to support SPARC

Christian Thalinger Christian.Thalinger at Sun.COM
Fri Apr 23 03:50:29 PDT 2010


On Wed, 2010-04-21 at 12:32 -0700, John Rose wrote:
> > The switch in load_sized_value is too clever for my taste.  How
> about just a switch on size_in_bytes with a test on is_signed where
> it's needed?
> 
> My high school writing teacher used to direct us, if we thought we had
> written something especially clever, to be ready to strike it out.
> Christian, if you break up the Overly Clever Switch in the sensible
> way Tom suggests, please break up the parallel one in the x86 code
> too.

I changed in both places.  -- Christian



More information about the hotspot-compiler-dev mailing list