review for 7042052: Xcomp crash with PopSynchronousTest

[Tom Rodriguez]

Thanks Igor and Vladimir.

tom

On May 4, 2011, at 5:34 PM, Igor Veresov wrote:

> 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.
>> 
>