review for 7042052: Xcomp crash with PopSynchronousTest

Igor Veresov igor.veresov at oracle.com
Wed May 4 17:34:24 PDT 2011


Looks good.

igor

On 5/4/11 5:09 PM, Tom Rodriguez wrote:
> http://cr.openjdk.java.net/~never/7042052
>
> 7042052: Xcomp crash with PopSynchronousTest
> Reviewed-by:
>
> In the fix for 7009361 I added some logic in deoptimization to adjust
> interpreted callers by the size of the signature at the call site
> instead of the size of the callee since they can be different in the
> presence of invokedynamic.  The logic for computing the arguments was
> missing some parens so it grouped incorrecty returning either 1 or 0.
> We have so much extra space in our frames most of the time that
> nothing broke but luckily this test case showed the problem.  Tested
> with failing test case from report.
>



More information about the hotspot-compiler-dev mailing list