Integrated: 8352914: Shenandoah: Change definition of ShenandoahSharedValue to int32_t to leverage platform atomics

Ben Taylor btaylor at openjdk.org
Tue Dec 9 00:20:20 UTC 2025


On Fri, 5 Dec 2025 18:44:08 GMT, Ben Taylor <btaylor at openjdk.org> wrote:

> The `STATIC_ASSERT` below this typedef appears to be out of date. 
> 
> The barriers check thread local copy of gc state, which is stored in `ShenandoahThreadLocalData::_gc_state`  and is type `char`, so the size requirement described by the assert is maintained even after this change.
> 
> Change passes all tier1 tests locally when run with Shenandoah GC.

This pull request has now been integrated.

Changeset: b86b2cbc
Author:    Ben Taylor <btaylor at openjdk.org>
Committer: Y. Srinivas Ramakrishna <ysr at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/b86b2cbc7d9dd57aeaf64f70f248a120ae3cb751
Stats:     5 lines in 1 file changed: 0 ins; 4 del; 1 mod

8352914: Shenandoah: Change definition of ShenandoahSharedValue to int32_t to leverage platform atomics

Reviewed-by: wkemper, ysr

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

PR: https://git.openjdk.org/jdk/pull/28681


More information about the shenandoah-dev mailing list