Merge pull request #45483 from aloubyansky/disable-c2-graalvm

Disable C2 for GraalVM as well in dev mode by default
quarkusio · Jan 10, 2025 · 6742705 · 6742705
2 parents 8ba65a8 + 539929a
commit 6742705
Show file tree

Hide file tree

Showing 2 changed files with 1 addition and 3 deletions.
diff --git a/core/deployment/src/main/java/io/quarkus/deployment/dev/DevModeCommandLineBuilder.java b/core/deployment/src/main/java/io/quarkus/deployment/dev/DevModeCommandLineBuilder.java
@@ -412,7 +412,6 @@ public DevModeCommandLine build() throws Exception {
 
         // if the --enable-preview flag was set, then we need to enable it when launching dev mode as well
         if (devModeContext.isEnablePreview()) {
-            System.out.println("ADDING enable-preview");
             jvmOptionsBuilder.add("enable-preview");
         }
 
@@ -516,7 +515,7 @@ private boolean isDisableC2(Map<String, List<ArtifactKey>> lockedJvmOptions) {
             extensionsEnableC2Warning(lockingExtensions);
             return false;
         }
-        return !JavaVersionUtil.isGraalvmJdk();
+        return true;
     }
 
     private void configureDebugging() throws Exception {

diff --git a/core/deployment/src/test/java/io/quarkus/deployment/dev/DevModeCommandLineBuilderTest.java b/core/deployment/src/test/java/io/quarkus/deployment/dev/DevModeCommandLineBuilderTest.java
@@ -29,7 +29,6 @@ public void extensionSetsJvmOptionWoValue() throws Exception {
                 Set.of());
 
         var args = getCliArguments(acmeMagic);
-        System.out.println(args);
         assertThat(args).contains("--enable-preview");
     }