RFR: JDK-8316756: C2 EA fails with "missing memory path" when encountering unsafe_arraycopy stub call [v3]

Tobias Holenstein tholenstein at openjdk.org
Mon Jan 15 15:55:22 UTC 2024


On Fri, 12 Jan 2024 17:21:35 GMT, Vladimir Kozlov <kvn at openjdk.org> wrote:

> > > > Flag setting (StoreB nodes) in JavaThread::_doing_unsafe_access is also not affected but it is volatile field and these stores should be staying where they are. They can't go up or down.
> > > 
> > > 
> > > In a private discussion with @tobiasholenstein, I proposed to move those stores into the corresponding stub. It would complicate the implementation a bit (platform-specific vs cross-platform implementation), but simplify things on IR level.
> > 
> > 
> > So you think we should go for that solution instead of this fix?
> 
> Yes. You may still need to fix EA to recognize RAW memory for `unsafe_arraycopy`.

I applied your patch for EA and added the test. Thanks!

tier1-4 pass

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

PR Comment: https://git.openjdk.org/jdk/pull/17347#issuecomment-1892420763


More information about the hotspot-compiler-dev mailing list