RFR: 8241605: Shenandoah: More aggressive reference discovery

[Aleksey Shipilev]

On 3/25/20 6:29 PM, Roman Kennke wrote:
> Webrev:
> http://cr.openjdk.java.net/~rkennke/JDK-8241605/webrev.00/

*) Feels like this cast:

1578     if (((ShenandoahEnqueueBarrierNode*)barrier)->can_eliminate(phase)) {

...should be done when we poll the node a few lines above?

1573     Node* barrier = state->enqueue_barrier(i);

*) "test_heap_stable" is misleading name now. "test_heap_state"? You already "assert
is_heap_state_test".

*) I actually wonder if checking for TRAVERSAL|MARKING should be done as another generic
optimization? It seems good by itself to test for this instead of falling through for
evac/update-refs? (Actually, what checks for *that* in the old code?)

It is good for sh/jdk sandbox. Please make sure you don't use bug ID when pushing there, so that
history would separate this prototype and the actual upstream change?

-- 
Thanks,
-Aleksey