review (XS) for 6981773: incorrect fill value with OptimizeFill

Christian Thalinger christian.thalinger at oracle.com
Thu Sep 2 00:01:02 PDT 2010


On Wed, 2010-09-01 at 19:56 -0700, Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/6981773
> 
> 6981773: incorrect fill value with OptimizeFill
> Reviewed-by:
> 
> The sparc assembly for the new fill logic doesn't fully initialize the
> value register in the check_fill_8_bytes path resulting in
> incorrectly filled arrays for certain lengths.  Tested with failing
> tests from nightly.
> 
> src/cpu/sparc/vm/stubGenerator_sparc.cpp

Looks good.  -- Christian



More information about the hotspot-compiler-dev mailing list