RFR 8233027: OopMapSet::all_do does oms.next() twice during iteration

[Vladimir Kozlov]

+1

Please, push at least tier1 testing to confirm that it solves issue and push it.

Thanks,
Vladimir

On 10/25/19 10:36 AM, Aleksey Shipilev wrote:
> On 10/25/19 7:14 PM, Tom Rodriguez wrote:
>> http://cr.openjdk.java.net/~never/8233027/webrev
>> https://bugs.openjdk.java.net/browse/JDK-8233027
>>
>> The fix for 8232083 introduced an extra next call in the derived pointers code.  This removes it.
> 
> Looks good. Plus, looks trivial.
> 
> I have been testing this patch for the last two hours, and it seems to resolve weird, random,
> intermittent C2/OSR failures we have seen with Shenandoah today.
>