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