RFR: 8240873: Shenandoah: Short-cut arraycopy barriers
Aleksey Shipilev
shade at redhat.com
Thu Mar 12 11:04:42 UTC 2020
On 3/12/20 12:04 PM, Roman Kennke wrote:
>> *) Wouldn't it make more sense to do acquire/release in (get|set)_update_watermark?
>
> Hmm ok. This requires making the field. I am not sure if the cast in
> get_update_watermark() is ok?
I don't quite understand why cast is needed. There are already _critical_pins and _live_data fields
that are atomic, why can't we do the same?
> http://cr.openjdk.java.net/~rkennke/JDK-8240873/webrev.02/
Looks fine, modulo nit above.
--
Thanks,
-Aleksey
More information about the hotspot-gc-dev
mailing list