Integrated: 8324050: Issue store-store barrier after re-materializing objects during deoptimization

Vladimir Kozlov kvn at openjdk.org
Mon Jan 22 22:53:30 UTC 2024


On Fri, 19 Jan 2024 22:00:15 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

> Added missing store-store barrier when we re-materialize scalar replaced object during deoptimization.
> 
> I also removed redundant `#if COMPILER2_OR_JVMCI` guards which were leftover from [JDK-8312579](https://bugs.openjdk.org/browse/JDK-8312579) changes. It added Vector API support to Graal and changed `#ifdef COMPILER2` to these `#if`. But this code is already under these `ifs`.
> 
> Tested tier1-3, scope, stress.
> 
> No new regression test.  I think it is "almost" impossible to hit this issue because there is a lot of VM's runtime code between the code which rematerialize scalar-replaced objects during deoptimization and a code in Interpreter which is executed after deoptimization and which may execute a store instruction that makes these objects accessible by other threads.

This pull request has now been integrated.

Changeset: 52523d33
Author:    Vladimir Kozlov <kvn at openjdk.org>
URL:       https://git.openjdk.org/jdk/commit/52523d33dde797bf03b15a05bb227b19b22c06be
Stats:     5 lines in 1 file changed: 4 ins; 0 del; 1 mod

8324050: Issue store-store barrier after re-materializing objects during deoptimization

Reviewed-by: dlong, shade

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

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


More information about the hotspot-dev mailing list