search

JetBrains / JetBrainsRuntime

Runtime environment based on OpenJDK for running IntelliJ Platform-based products on Windows, macOS, and Linux
GNU General Public License v2.0
1.21k stars 188 forks source link

JBR-7351 DCEVM: Respect explicit -XX:+ClassUnloading option #411

Closed skybber closed 1 week ago

skybber commented 1 week ago

Ensure that ClassUnloading is not disabled if the -XX:+ClassUnloading flag is explicitly set by the user.

mkartashev commented 1 week ago

I'm not convinced that changing an undesirable, yet deterministic, behavior (code cache overflow) with even more undesirable (crash) and completely non-deterministic (may happen at random) is worth doing. Specifying -XX:+ClassUnloading does in no way imply that JVM might become unstable. Maybe create a totally new flag just for this purpose, but again, if it still makes the JVM unstable, why use it at all?

skybber commented 1 week ago

Merged manually.