[GR-70564] Fix native-image-configure to properly register types from JNI sections as JNI accessed

PullRequest: graal/22330

Author: loicottet

substratevm/src/com.oracle.svm.configure/src/com/oracle/svm/configure/LegacyReflectionConfigurationParser.java

@@ -97,6 +97,9 @@ protected void parseClass(EconomicMap<String, Object> data) {
         C queryCondition = isType ? conditionResolver.alwaysTrue() : condition;
         T clazz = result.get();
         delegate.registerType(conditionResult.get(), clazz);
+        if (jniParser) {
+            delegate.registerAsJniAccessed(condition, clazz);
+        }
 
         registerIfNotDefault(data, false, clazz, "allDeclaredConstructors", () -> delegate.registerDeclaredConstructors(condition, false, jniParser, clazz));
         registerIfNotDefault(data, false, clazz, "allPublicConstructors", () -> delegate.registerPublicConstructors(condition, false, jniParser, clazz));

substratevm/src/com.oracle.svm.configure/src/com/oracle/svm/configure/ReflectionMetadataParser.java

@@ -106,6 +106,8 @@ protected void parseClass(EconomicMap<String, Object> data) {
             if (!jniParser) {
                 registerIfNotDefault(data, false, clazz, "serializable", () -> delegate.registerAsSerializable(condition, clazz));
                 registerIfNotDefault(data, false, clazz, "jniAccessible", () -> delegate.registerAsJniAccessed(condition, clazz));
+            } else {
+                delegate.registerAsJniAccessed(condition, clazz);
             }
 
             registerIfNotDefault(data, false, clazz, "allDeclaredConstructors", () -> delegate.registerDeclaredConstructors(condition, false, typeJniAccessible, clazz));

substratevm/src/com.oracle.svm.hosted/src/com/oracle/svm/hosted/config/JNIRegistryAdapter.java

@@ -140,11 +140,6 @@ public void registerAsSerializable(AccessCondition condition, Class<?> clazz) {
         VMError.shouldNotReachHere("serializable cannot be set on JNI registrations");
     }
 
-    @Override
-    public void registerAsJniAccessed(AccessCondition condition, Class<?> clazz) {
-        VMError.shouldNotReachHere("jniAccessible cannot be set on JNI registrations");
-    }
-
     private static void ensureJniAccessible(boolean jniAccessible) {
         VMError.guarantee(jniAccessible, "JNIRegistryAdapter can only be used for JNI queries");
     }