Refactor ReadMemberNode so it can be DSL inlinable

Author: horakivo

src/main/java/org/truffleruby/interop/InteropNodes.java

@@ -1269,25 +1269,21 @@ protected Object members(Object receiver, boolean internal,
 
     }
 
-    @GenerateUncached
-    @GenerateNodeFactory
     @CoreMethod(names = "read_member", onSingleton = true, required = 2)
-    @NodeChild(value = "argumentNodes", type = RubyNode[].class)
-    public abstract static class ReadMemberNode extends RubySourceNode {
+    public abstract static class InteropReadMemberNode extends CoreMethodArrayArgumentsNode {
 
-        @NeverDefault
-        public static ReadMemberNode create() {
-            return InteropNodesFactory.ReadMemberNodeFactory.create(null);
+        @Specialization
+        protected Object readMember(Object receiver, Object identifier,
+                @Cached ReadMemberNode readMemberNode) {
+            return readMemberNode.execute(receiver, identifier);
         }
+    }
 
-        public static ReadMemberNode create(RubyNode[] argumentNodes) {
-            return InteropNodesFactory.ReadMemberNodeFactory.create(argumentNodes);
-        }
+    @GenerateUncached
+    public abstract static class ReadMemberNode extends RubyBaseNode {
 
         public abstract Object execute(Object receiver, Object identifier);
 
-        abstract RubyNode[] getArgumentNodes();
-
         @Specialization(limit = "getInteropCacheLimit()")
         protected Object readMember(Object receiver, Object identifier,
                 @CachedLibrary("receiver") InteropLibrary receivers,
@@ -1298,12 +1294,6 @@ protected Object readMember(Object receiver, Object identifier,
             final Object foreign = InteropNodes.readMember(receivers, receiver, name, translateInteropException);
             return foreignToRubyNode.executeConvert(foreign);
         }
-
-        @Override
-        public RubyNode cloneUninitialized() {
-            return create(cloneUninitialized(getArgumentNodes())).copyFlags(this);
-        }
-
     }
 
     @GenerateUncached