LocalVariableGetNode is DSL inlinable node

Author: horakivo

src/main/java/org/truffleruby/core/binding/BindingNodes.java

@@ -278,35 +278,37 @@ protected Object localVariableGet(RubyBinding binding, Object nameObject,
                 @Cached NameToJavaStringNode nameToJavaStringNode,
                 @Cached LocalVariableGetNode localVariableGetNode) {
             final var name = nameToJavaStringNode.execute(this, nameObject);
-            return localVariableGetNode.execute(binding, name);
+            return localVariableGetNode.execute(this, binding, name);
         }
     }
 
     @GenerateUncached
     @ImportStatic(BindingNodes.class)
+    @GenerateInline
+    @GenerateCached(false)
     public abstract static class LocalVariableGetNode extends RubyBaseNode {
 
-        public abstract Object execute(RubyBinding binding, String name);
+        public abstract Object execute(Node node, RubyBinding binding, String name);
 
         @Specialization(guards = "!isHiddenVariable(name)")
-        protected Object localVariableGet(RubyBinding binding, String name,
+        protected static Object localVariableGet(Node node, RubyBinding binding, String name,
                 @Cached FindAndReadDeclarationVariableNode readNode) {
             MaterializedFrame frame = binding.getFrame();
             Object result = readNode.execute(frame, name, null);
             if (result == null) {
                 throw new RaiseException(
-                        getContext(),
-                        coreExceptions().nameErrorLocalVariableNotDefined(name, binding, this));
+                        getContext(node),
+                        coreExceptions(node).nameErrorLocalVariableNotDefined(name, binding, node));
             }
             return result;
         }
 
         @TruffleBoundary
         @Specialization(guards = "isHiddenVariable(name)")
-        protected Object localVariableGetLastLine(RubyBinding binding, String name) {
+        protected static Object localVariableGetLastLine(Node node, RubyBinding binding, String name) {
             throw new RaiseException(
-                    getContext(),
-                    coreExceptions().nameError("Bad local variable name", binding, name, this));
+                    getContext(node),
+                    coreExceptions(node).nameError("Bad local variable name", binding, name, node));
         }
     }
 

src/main/java/org/truffleruby/debug/BindingLocalVariablesObject.java

@@ -10,7 +10,9 @@
 package org.truffleruby.debug;
 
 import com.oracle.truffle.api.CompilerDirectives.CompilationFinal;
+import com.oracle.truffle.api.dsl.Bind;
 import com.oracle.truffle.api.library.CachedLibrary;
+import com.oracle.truffle.api.nodes.Node;
 import org.truffleruby.core.binding.BindingNodes;
 import org.truffleruby.core.binding.RubyBinding;
 import org.truffleruby.core.string.StringUtils;
@@ -55,10 +57,11 @@ protected Object getMembers(boolean includeInternal) {
 
     @ExportMessage
     protected Object readMember(String member,
-            @Cached @Exclusive BindingNodes.LocalVariableGetNode localVariableGetNode)
+            @Cached @Exclusive BindingNodes.LocalVariableGetNode localVariableGetNode,
+            @Bind("$node") Node node)
             throws UnknownIdentifierException {
         try {
-            return localVariableGetNode.execute(binding, member);
+            return localVariableGetNode.execute(node, binding, member);
         } catch (RaiseException e) {
             throw UnknownIdentifierException.create(member);
         }

src/main/java/org/truffleruby/debug/RubyScope.java

@@ -11,6 +11,7 @@
 
 import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
 import com.oracle.truffle.api.TruffleLanguage;
+import com.oracle.truffle.api.dsl.Bind;
 import com.oracle.truffle.api.dsl.Cached;
 import com.oracle.truffle.api.dsl.Cached.Exclusive;
 import com.oracle.truffle.api.dsl.Specialization;
@@ -22,6 +23,7 @@
 import com.oracle.truffle.api.library.CachedLibrary;
 import com.oracle.truffle.api.library.ExportLibrary;
 import com.oracle.truffle.api.library.ExportMessage;
+import com.oracle.truffle.api.nodes.Node;
 import com.oracle.truffle.api.nodes.RootNode;
 import com.oracle.truffle.api.source.SourceSection;
 import org.truffleruby.RubyContext;
@@ -125,10 +127,11 @@ protected static Object readSelf(RubyScope scope, String member) {
 
         @Specialization(guards = "!RECEIVER_MEMBER.equals(member)")
         protected static Object read(RubyScope scope, String member,
-                @Cached @Exclusive BindingNodes.LocalVariableGetNode localVariableGetNode)
+                @Cached @Exclusive BindingNodes.LocalVariableGetNode localVariableGetNode,
+                @Bind("this") Node node)
                 throws UnknownIdentifierException {
             try {
-                return localVariableGetNode.execute(scope.binding, member);
+                return localVariableGetNode.execute(node, scope.binding, member);
             } catch (RaiseException e) {
                 throw UnknownIdentifierException.create(member);
             }