RFR: 8354523: runtime/Monitor/SyncOnValueBasedClassTest.java triggers SIGSEGV

Wed Apr 16 16:27:57 UTC 2025

On Wed, 16 Apr 2025 12:28:22 GMT, Martin Doerr <mdoerr at openjdk.org> wrote:

> > this would bring DiagnoseSyncOnValueBasedClasses-handling into the native entry in sharedRuntime_x86_64.cpp
> 
> Sounds like a bug that it is currently missing?
> 
> Ah, seems like Value-based Classes are not allowed to use `synchronized`. But, would that hurt?

You mean if it would hurt doing the check also in the native wrapper, right? It would not hurt, I'd say. It would be just as redundant as on the other code paths entering a synchronized method where it wasn't worth the effort making the distinction.

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

PR Comment: https://git.openjdk.org/jdk/pull/24660#issuecomment-2810099939