[GR-31478] Force linking of types resolved by GraalTruffleRuntime.

PullRequest: graal/8977

Author: DSouzaM

common.json

@@ -6,8 +6,8 @@
     "oraclejdk8":         {"name": "oraclejdk", "version": "8u301+03-jvmci-21.2-b01", "platformspecific": true },
     "oraclejdk8Debug":    {"name": "oraclejdk", "version": "8u301+03-jvmci-21.2-b01-fastdebug", "platformspecific": true },
 
-    "openjdk11":          {"name": "openjdk",   "version": "11.0.3+7", "platformspecific": true },
-    "oraclejdk11":        {"name": "oraclejdk", "version": "11.0.6+8", "platformspecific": true },
+    "openjdk11":          {"name": "openjdk",   "version": "11.0.11+9", "platformspecific": true },
+    "oraclejdk11":        {"name": "oraclejdk", "version": "11.0.11+9", "platformspecific": true },
     "labsjdk-ce-11":      {"name": "labsjdk",   "version": "ce-11.0.11+8-jvmci-21.2-b01", "platformspecific": true },
     "labsjdk-ee-11":      {"name": "labsjdk",   "version": "ee-11.0.11+9-jvmci-21.2-b01", "platformspecific": true },
 

compiler/src/org.graalvm.compiler.truffle.runtime/src/org/graalvm/compiler/truffle/runtime/GraalTruffleRuntime.java

@@ -431,7 +431,11 @@ public ResolvedJavaType resolveType(MetaAccessProvider metaAccess, String classN
             }
             throw new NoClassDefFoundError(className);
         }
-        return metaAccess.lookupJavaType(c);
+        ResolvedJavaType type = metaAccess.lookupJavaType(c);
+        // In some situations, we may need the class to be linked now, especially if we are
+        // compiling immediately (e.g., to successfully devirtualize FrameWithoutBoxing methods).
+        type.link();
+        return type;
     }
 
     protected void installDefaultListeners() {

substratevm/src/com.oracle.svm.graal/src/com/oracle/svm/graal/hosted/GraalObjectReplacer.java

@@ -31,6 +31,7 @@
 import java.util.Map;
 import java.util.function.Function;
 
+import com.oracle.svm.core.util.VMError;
 import org.graalvm.compiler.api.replacements.SnippetReflectionProvider;
 import org.graalvm.compiler.api.runtime.GraalRuntime;
 import org.graalvm.compiler.core.common.spi.ConstantFieldProvider;
@@ -284,6 +285,7 @@ public synchronized SubstrateType createType(JavaType original) {
         }
 
         AnalysisType aType = toAnalysisType(original);
+        VMError.guarantee(aType.isLinked(), "types reachable for JIT compilation must not have linkage errors");
         SubstrateType sType = types.get(aType);
 
         if (sType == null) {

substratevm/src/com.oracle.svm.graal/src/com/oracle/svm/graal/meta/SubstrateType.java

@@ -428,12 +428,12 @@ public ResolvedJavaMethod getClassInitializer() {
 
     @Override
     public boolean isLinked() {
-        throw VMError.unimplemented();
+        return true;  // types are always linked
     }
 
     @Override
     public void link() {
-        throw VMError.unimplemented();
+        // do nothing
     }
 
     @Override
@@ -498,8 +498,8 @@ public com.oracle.truffle.api.nodes.NodeFieldAccessor[] getFields() {
     @SuppressWarnings("deprecation")
     @Override
     public com.oracle.truffle.api.nodes.NodeFieldAccessor getParentField() {
-        return SubstrateNodeFieldAccessor
-                        .fromSubstrateField(getNodeFields(field -> DynamicHub.toClass(field.getDeclaringClass().getHub()) == Node.class && field.getName().equals("parent")).iterator().next());
+        return SubstrateNodeFieldAccessor.fromSubstrateField(
+                        getNodeFields(field -> DynamicHub.toClass(field.getDeclaringClass().getHub()) == Node.class && field.getName().equals("parent")).iterator().next());
     }
 
     @SuppressWarnings("deprecation")