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

[Aleksey Shipilev]

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.

-- 
Thanks,
-Aleksey