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

[Roman Kennke]

On Wed, 16 Apr 2025 14:37:58 GMT, Roman Kennke <rkennke at openjdk.org> wrote:

> I wonder if it would be better to zero the cache in InterpreterRuntime::monitorenter and have a C1 entry rather than going directly to monitor_enter_helper. Then we can zero the BasicLock::monitor_cache in the two places where it shouldn't be set.

I pushed a change that clears the cache in the C1 and interpreter entries. It's certainly the simplest solution but kinda itches the perfectionist in me: we still need to clear the cache in the fast-paths anyway, so we may as well do it before DiagnoseSyncOnValueBasedClasses-block?

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

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