RFR: 8305351: C2 setScopedValueCache intrinsic doesn't use access API

Martin Doerr mdoerr at openjdk.org
Thu Apr 13 09:52:35 UTC 2023


On Tue, 4 Apr 2023 12:40:14 GMT, Erik Österlund <eosterlund at openjdk.org> wrote:

> The setScopedValueCache intrinsic for C2 doesn't use the access API. Instead, we store into an OopHandle with a raw store. That doesn't necessarily play well with all GCs, for example Shenandoah and generational ZGC. We should use the access API to ensure the right barriers are emitted.

Thanks for fixing it! I guess we should also backport the fix for Shenandoah.

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

Marked as reviewed by mdoerr (Reviewer).

PR Review: https://git.openjdk.org/jdk/pull/13324#pullrequestreview-1383059862


More information about the hotspot-compiler-dev mailing list