RFR: 8254315: Shenandoah: Concurrent weak reference processing [v12]

[Roman Kennke]

On Tue, 27 Oct 2020 13:57:28 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

>> src/hotspot/share/gc/shenandoah/c2/shenandoahBarrierSetC2.cpp line 1062:
>> 
>>> 1060:     Node* in2 = n->in(2);
>>> 1061:     if (in1->bottom_type() == TypePtr::NULL_PTR &&
>>> 1062:         (in2->Opcode() != Op_ShenandoahLoadReferenceBarrier ||
>> 
>> This is a bugfix, right? It changes `in1` (seemingly incorrect) to `in2` (seemingly correct). If so, maybe we should split it out to fix previous releases too?
>
> Actually I think I *introduced* a bug there. It seems curious that it worked that way :-) I'm fixing it.

You are right. Tracking & fixing this here: https://bugs.openjdk.java.net/browse/JDK-8255534

-------------

PR: https://git.openjdk.java.net/jdk/pull/505