RFR: 8352914: Shenandoah: Change definition of ShenandoahSharedValue to int32_t to leverage platform atomics
duke
duke at openjdk.org
Mon Dec 8 23:29:59 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.
@benty-amzn
Your change (at version 90923ab3b090ae4021bcb4bf47076f6124cd2491) is now ready to be sponsored by a Committer.
-------------
PR Comment: https://git.openjdk.org/jdk/pull/28681#issuecomment-3629465464
More information about the shenandoah-dev
mailing list