Introduce IsolatedTruffleCodeInstallBridge to avoid truffle dependency

Implements `OptimizedAssumptionDependency` on top of
`IsolatedCodeInstallBridge` in `com.oracle.svm.truffle` to avoid pulling
truffle dependencies in `com.oracle.svm.graal`.

Author: zakkak

substratevm/src/com.oracle.svm.graal/src/com/oracle/svm/graal/isolated/IsolatedCodeInstallBridge.java

@@ -24,20 +24,16 @@
  */
 package com.oracle.svm.graal.isolated;
 
-import org.graalvm.compiler.truffle.common.CompilableTruffleAST;
-import org.graalvm.compiler.truffle.common.OptimizedAssumptionDependency;
-
 import com.oracle.svm.core.deopt.SubstrateInstalledCode;
 import com.oracle.svm.core.util.VMError;
 
 import jdk.vm.ci.code.InstalledCode;
 
 /**
  * A helper to pass information for installing code in the compilation client through a Truffle
- * compilation. It does not implement {@link InstalledCode} or {@link OptimizedAssumptionDependency}
- * in any meaningful way.
+ * compilation. It does not implement {@link InstalledCode} in any meaningful way.
  */
-public final class IsolatedCodeInstallBridge extends InstalledCode implements OptimizedAssumptionDependency {
+public class IsolatedCodeInstallBridge extends InstalledCode {
     private final ClientHandle<? extends SubstrateInstalledCode.Factory> factoryHandle;
     private ClientHandle<? extends SubstrateInstalledCode> installedCodeHandle;
 
@@ -59,7 +55,7 @@ public ClientHandle<? extends SubstrateInstalledCode> getSubstrateInstalledCodeH
         return installedCodeHandle;
     }
 
-    private static final String DO_NOT_CALL_REASON = IsolatedCodeInstallBridge.class.getSimpleName() +
+    protected static final String DO_NOT_CALL_REASON = IsolatedCodeInstallBridge.class.getSimpleName() +
                     " only acts as an accessor for cross-isolate data. None of the implemented methods may be called.";
 
     @Override
@@ -87,19 +83,9 @@ public byte[] getCode() {
         throw VMError.shouldNotReachHere(DO_NOT_CALL_REASON);
     }
 
-    @Override
-    public void onAssumptionInvalidated(Object source, CharSequence reason) {
-        throw VMError.shouldNotReachHere(DO_NOT_CALL_REASON);
-    }
-
     @Override
     public Object executeVarargs(Object... args) {
         throw VMError.shouldNotReachHere(DO_NOT_CALL_REASON);
     }
 
-    @Override
-    public CompilableTruffleAST getCompilable() {
-        throw VMError.shouldNotReachHere(DO_NOT_CALL_REASON);
-    }
-
 }

substratevm/src/com.oracle.svm.truffle/src/com/oracle/svm/truffle/isolated/IsolatedCompilableTruffleAST.java

@@ -40,7 +40,6 @@
 import com.oracle.svm.graal.isolated.ClientIsolateThread;
 import com.oracle.svm.graal.isolated.CompilerHandle;
 import com.oracle.svm.graal.isolated.CompilerIsolateThread;
-import com.oracle.svm.graal.isolated.IsolatedCodeInstallBridge;
 import com.oracle.svm.graal.isolated.IsolatedCompileClient;
 import com.oracle.svm.graal.isolated.IsolatedCompileContext;
 import com.oracle.svm.graal.isolated.IsolatedObjectConstant;
@@ -150,7 +149,7 @@ public SubstrateInstalledCode createSubstrateInstalledCode() {
 
     @Override
     public InstalledCode createPreliminaryInstalledCode() {
-        return new IsolatedCodeInstallBridge(handle);
+        return new IsolatedTruffleCodeInstallBridge(handle);
     }
 
     @Override

substratevm/src/com.oracle.svm.truffle/src/com/oracle/svm/truffle/isolated/IsolatedTruffleCodeInstallBridge.java

@@ -0,0 +1,30 @@
+package com.oracle.svm.truffle.isolated;
+
+import com.oracle.svm.core.deopt.SubstrateInstalledCode;
+import com.oracle.svm.core.util.VMError;
+import com.oracle.svm.graal.isolated.ClientHandle;
+import com.oracle.svm.graal.isolated.IsolatedCodeInstallBridge;
+import jdk.vm.ci.code.InstalledCode;
+import org.graalvm.compiler.truffle.common.CompilableTruffleAST;
+import org.graalvm.compiler.truffle.common.OptimizedAssumptionDependency;
+
+/**
+ * A helper to pass information for installing code in the compilation client through a Truffle
+ * compilation. It does not implement {@link InstalledCode} or {@link OptimizedAssumptionDependency}
+ * in any meaningful way.
+ */
+public final class IsolatedTruffleCodeInstallBridge extends IsolatedCodeInstallBridge implements OptimizedAssumptionDependency {
+    public IsolatedTruffleCodeInstallBridge(ClientHandle<? extends SubstrateInstalledCode.Factory> factoryHandle) {
+        super(factoryHandle);
+    }
+
+    @Override
+    public void onAssumptionInvalidated(Object source, CharSequence reason) {
+        throw VMError.shouldNotReachHere(DO_NOT_CALL_REASON);
+    }
+
+    @Override
+    public CompilableTruffleAST getCompilable() {
+        throw VMError.shouldNotReachHere(DO_NOT_CALL_REASON);
+    }
+}