If a call target supplier is fulfilled, just use it

Co-authored-by: Matthew Alt <[email protected]>

Author: chrisseaton

src/main/java/org/truffleruby/collections/CachedSupplier.java

@@ -22,7 +22,7 @@ public CachedSupplier(Supplier<T> supplier) {
 
     @Override
     public T get() {
-        if (value != null) {
+        if (isAvailable()) {
             return value;
         }
         synchronized (this) {
@@ -33,4 +33,9 @@ public T get() {
             return value;
         }
     }
+
+    public boolean isAvailable() {
+        return value != null;
+    }
+
 }

src/main/java/org/truffleruby/language/methods/InternalMethod.java

@@ -171,6 +171,13 @@ private InternalMethod(
         this.proc = proc;
         this.callTarget = callTarget;
         this.callTargetSupplier = callTargetSupplier;
+
+        /* If the call target supplier has already been run, then don't wait until the first time the InternalMethod is
+         * asked for the call target, because this would be a deoptimisation in getCallTarget. */
+
+        if (callTarget == null && callTargetSupplier != null && callTargetSupplier.isAvailable()) {
+            this.callTarget = callTargetSupplier.get();
+        }
     }
 
     public SharedMethodInfo getSharedMethodInfo() {