RFR: 8288282: Zero-release build is broken after JDK-8279047 due to UseHeavyMonitors is read-only [v2]
Aleksey Shipilev
shade at openjdk.java.net
Mon Jun 13 09:45:51 UTC 2022
On Mon, 13 Jun 2022 09:38:43 GMT, Jie Fu <jiefu at openjdk.org> wrote:
> > Let's not force VM into different option settings in `release` and `fastdebug`, that's just calling for trouble.
>
> So what's the trouble would be?
> There are many VM flags which are only available for debug VMs.
Those flags are usually having the same value for release and debug VMs. Your current patch changes the flag values depending on VM mode, which exposes potentially different behaviors depending on VM mode. Imagine chasing a bug in `release` VM that does not reproduce in `debug` VM, those are not fun. Let's not do that. It would be all around easier and safer. to just disable to flag in all VM modes.
The block your patch is modifying is added by me in JDK-8273486, I say we just redo it without involving `UseHeavyMonitors` at all, and just disable the test that is affected by it.
-------------
PR: https://git.openjdk.org/jdk/pull/9138
More information about the hotspot-dev
mailing list