[GR-65209] Use RuntimeJNIAccessSupport only when JNI is set.

PullRequest: graal/20867

Author: vjovanov

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/NativeImageGenerator.java

@@ -1112,7 +1112,9 @@ protected void setupNativeImage(String imageName, OptionValues options, Map<Meth
                                 new SubstrateClassInitializationPlugin(hostVM), this.isStubBasedPluginsSupported(), aProviders);
 
                 loader.classLoaderSupport.getClassesToIncludeUnconditionally().forEach(cls -> bb.registerTypeForBaseImage(cls));
-                PreserveOptionsSupport.registerPreservedClasses(loader.classLoaderSupport);
+                if (loader.classLoaderSupport.isPreserveMode()) {
+                    PreserveOptionsSupport.registerPreservedClasses(loader.classLoaderSupport);
+                }
 
                 registerEntryPointStubs(entryPoints);
             }

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/image/PreserveOptionsSupport.java

@@ -150,7 +150,6 @@ public static void registerPreservedClasses(NativeImageClassLoaderSupport classL
                         .toList();
 
         final RuntimeReflectionSupport reflection = ImageSingletons.lookup(RuntimeReflectionSupport.class);
-        final RuntimeJNIAccessSupport jni = ImageSingletons.lookup(RuntimeJNIAccessSupport.class);
         final RuntimeProxyCreationSupport proxy = ImageSingletons.lookup(RuntimeProxyCreationSupport.class);
         final ConfigurationCondition always = ConfigurationCondition.alwaysTrue();
 
@@ -178,20 +177,29 @@ public static void registerPreservedClasses(NativeImageClassLoaderSupport classL
                 proxy.addProxyClass(always, c);
             }
 
-            jni.register(ConfigurationCondition.alwaysTrue(), c);
             try {
-                for (Method declaredMethod : c.getDeclaredMethods()) {
-                    jni.register(always, false, declaredMethod);
-                }
-                for (Constructor<?> declaredConstructor : c.getDeclaredConstructors()) {
-                    jni.register(always, false, declaredConstructor);
-                }
                 for (Field declaredField : c.getDeclaredFields()) {
-                    jni.register(always, false, declaredField);
                     reflection.register(always, false, declaredField);
                 }
             } catch (LinkageError e) {
-                /* If we can't link we can not register for JNI and reflection */
+                /* If we can't link we can not register for reflection */
+            }
+            if (SubstrateOptions.JNI.getValue()) {
+                final RuntimeJNIAccessSupport jni = ImageSingletons.lookup(RuntimeJNIAccessSupport.class);
+                jni.register(always, c);
+                try {
+                    for (Method declaredMethod : c.getDeclaredMethods()) {
+                        jni.register(always, false, declaredMethod);
+                    }
+                    for (Constructor<?> declaredConstructor : c.getDeclaredConstructors()) {
+                        jni.register(always, false, declaredConstructor);
+                    }
+                    for (Field declaredField : c.getDeclaredFields()) {
+                        jni.register(always, false, declaredField);
+                    }
+                } catch (LinkageError e) {
+                    /* If we can't link we can not register for JNI and reflection */
+                }
             }
 
             // if we register as unsafe allocated earlier there are build-time