review for 7150051: incorrect oopmap in critical native

[Tom Rodriguez]

Thanks, Chris and Vladimir.

tom

On Mar 1, 2012, at 1:27 AM, Christian Thalinger wrote:

> Looks good.  -- Chris
> 
> On Mar 1, 2012, at 3:54 AM, Tom Rodriguez wrote:
> 
>> http://cr.openjdk.java.net/~never/7150051
>> 2 lines changed: 1 ins; 1 del; 0 mod; 3858 unchg
>> 
>> 7150051: incorrect oopmap in critical native
>> Reviewed-by:
>> 
>> The slot for the oopmap is being incremented before being used leading
>> to incorrect oopmaps when blocking for the critical native.  The fix
>> is to move the increment until after it's used.  Tested with failing
>> regression tests from report.
>> 
>