[GR-37493] Use @DenyReplace for non-adoptable nodes

Author: eregon

src/main/java/org/truffleruby/language/WarnNode.java

@@ -9,6 +9,7 @@
  */
 package org.truffleruby.language;
 
+import com.oracle.truffle.api.nodes.DenyReplace;
 import com.oracle.truffle.api.nodes.NodeCost;
 import org.truffleruby.RubyContext;
 import org.truffleruby.RubyLanguage;
@@ -75,16 +76,9 @@ private static String buildWarningMessage(SourceSection sourceSection, String me
         return sourceLocation + "warning: " + message;
     }
 
-    public static class UncachedWarnNode extends RubyBaseNode {
+    abstract static class AbstractUncachedWarnNode extends RubyBaseNode {
 
-        public static final UncachedWarnNode INSTANCE = new UncachedWarnNode();
-
-        UncachedWarnNode() {
-        }
-
-        public boolean shouldWarn() {
-            return coreLibrary().warningsEnabled();
-        }
+        public abstract boolean shouldWarn();
 
         public void warningMessage(SourceSection sourceSection, String message) {
             assert shouldWarn();
@@ -108,4 +102,17 @@ public boolean isAdoptable() {
 
     }
 
+    @DenyReplace
+    public static final class UncachedWarnNode extends AbstractUncachedWarnNode {
+
+        public static final UncachedWarnNode INSTANCE = new UncachedWarnNode();
+
+        UncachedWarnNode() {
+        }
+
+        public boolean shouldWarn() {
+            return coreLibrary().warningsEnabled();
+        }
+    }
+
 }

src/main/java/org/truffleruby/language/WarningNode.java

@@ -9,6 +9,7 @@
  */
 package org.truffleruby.language;
 
+import com.oracle.truffle.api.nodes.DenyReplace;
 
 /** Warns only if $VERBOSE is true. Corresponds to Kernel#warn(message, uplevel: 1) if $VERBOSE, but in Java with a
  * given SourceSection. */
@@ -20,7 +21,8 @@ public boolean shouldWarn() {
         return verbosity == Boolean.TRUE;
     }
 
-    public static class UncachedWarningNode extends UncachedWarnNode {
+    @DenyReplace
+    public static final class UncachedWarningNode extends AbstractUncachedWarnNode {
 
         public static final UncachedWarningNode INSTANCE = new UncachedWarningNode();
 

src/main/java/org/truffleruby/language/dispatch/DispatchNode.java

@@ -13,6 +13,7 @@
 import com.oracle.truffle.api.RootCallTarget;
 import com.oracle.truffle.api.frame.Frame;
 import com.oracle.truffle.api.interop.TruffleObject;
+import com.oracle.truffle.api.nodes.DenyReplace;
 import com.oracle.truffle.api.nodes.DirectCallNode;
 import com.oracle.truffle.api.nodes.NodeCost;
 import com.oracle.truffle.api.profiles.BranchProfile;
@@ -372,7 +373,8 @@ public final void applySplittingInliningStrategy(RootCallTarget callTarget, Stri
         }
     }
 
-    private static class Uncached extends DispatchNode {
+    @DenyReplace
+    private static final class Uncached extends DispatchNode {
 
         static final Uncached[] UNCACHED_NODES = new Uncached[DispatchConfiguration.values().length];
         static {

src/main/java/org/truffleruby/language/dispatch/InternalRespondToNode.java

@@ -10,6 +10,7 @@
 package org.truffleruby.language.dispatch;
 
 import com.oracle.truffle.api.frame.VirtualFrame;
+import com.oracle.truffle.api.nodes.DenyReplace;
 import com.oracle.truffle.api.nodes.NodeCost;
 import org.truffleruby.core.kernel.KernelNodes.RespondToNode;
 import org.truffleruby.core.klass.RubyClass;
@@ -71,7 +72,8 @@ public boolean execute(VirtualFrame frame, Object receiver, String methodName) {
         return method != null && method.isDefined() && method.isImplemented();
     }
 
-    private static class Uncached extends InternalRespondToNode {
+    @DenyReplace
+    private static final class Uncached extends InternalRespondToNode {
 
         static final Uncached[] UNCACHED_NODES = new Uncached[DispatchConfiguration.values().length];
         static {