RFR: 8277654: Shenandoah: Don't produce new memory state in C2 LRB runtime call [v2]
Roland Westrelin
roland at openjdk.java.net
Fri Nov 26 16:28:07 UTC 2021
On Fri, 26 Nov 2021 09:42:34 GMT, Roman Kennke <rkennke at openjdk.org> wrote:
>> The runtime call in expanded Shenandoah LRB doesn't need to produce (and consume) raw memory state. I believe this is a left-over from when the LRB (or WB) allocated from TLABs, and would mess with TLAB pointers, when not ordered correctly. (Also, we used to require ordering with RBs back when we had them, but we already removed that memory dependency on -8 offset)
>>
>> Testing:
>> - [x] hotspot_gc_shenandoah
>> - [x] specjvm -XX:+UseShenandoahGC
>> - [x] tier1
>> - [x] tier2
>> - [x] tier3
>
> Roman Kennke has updated the pull request incrementally with one additional commit since the last revision:
>
> Remove verify_raw_mem()
Looks good to me.
-------------
Marked as reviewed by roland (Reviewer).
PR: https://git.openjdk.java.net/jdk/pull/6526
More information about the hotspot-gc-dev
mailing list