Remove Truffle::Debug.yarp_*, they are redundant now that Prism is the default

Author: eregon

doc/contributor/how-to-guide.md

@@ -1139,24 +1139,6 @@ jt -q ruby -e 'puts Truffle::Debug.java_class_of([])'
 RubyArray
 ```
 
-### Truffle::Debug.yarp_serialize
-
-```
-jt -q ruby -e 'puts Truffle::Debug.yarp_serialize("1").dump'
-"YARP\x00\x04\x00@7\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00L\f\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00\x00S\x19\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x01\x00\x00\x00&\b\x00\x00\x00\x00\x00\x00\x00\x01\x00\x00\x00\x00"
-```
-
-### Truffle::Debug.yarp_parse
-
-```
-jt -q ruby -e 'puts Truffle::Debug.yarp_parse("[].to_ary")'
-ProgramNode
-  Scope
-  StatementsNode
-    CallNode
-      ArrayNode
-```
-
 ### Truffle::Debug.ast
 
 ```

spec/truffle/parsing/parsing_spec.rb

@@ -114,7 +114,7 @@
           end
 
           unless actual_ast == expected_ast
-            $stderr.puts "\n#{filename}\nYARP AST:", Truffle::Debug.yarp_parse(source_code)
+            $stderr.puts "\n#{filename}\nYARP AST:", Truffle::Debug.parse_ast(source_code)
           end
           actual_ast.should == expected_ast
         end

spec/truffle/yarp_spec.rb

@@ -69,11 +69,9 @@
 import org.truffleruby.language.CallStackManager;
 import org.truffleruby.language.ImmutableRubyObject;
 import org.truffleruby.core.string.ImmutableRubyString;
-import org.truffleruby.language.Nil;
 import org.truffleruby.language.RubyDynamicObject;
 import org.truffleruby.language.RubyGuards;
 import org.truffleruby.language.RubyRootNode;
-import org.truffleruby.language.arguments.NoKeywordArgumentsDescriptor;
 import org.truffleruby.language.arguments.RubyArguments;
 import org.truffleruby.language.backtrace.BacktraceFormatter;
 import org.truffleruby.language.library.RubyStringLibrary;
@@ -112,8 +110,6 @@
 import org.truffleruby.parser.RubySource;
 import org.truffleruby.parser.TranslatorEnvironment;
 import org.truffleruby.parser.YARPTranslatorDriver;
-import org.prism.Loader;
-import org.prism.Parser;
 
 @CoreModule("Truffle::Debug")
 public abstract class TruffleDebugNodes {
@@ -250,83 +246,6 @@ Object printBacktrace() {
 
     }
 
-    @CoreMethod(names = "yarp_serialize", onSingleton = true, required = 1)
-    public abstract static class YARPSerializeNode extends CoreMethodArrayArgumentsNode {
-        @TruffleBoundary
-        @Specialization(guards = "strings.isRubyString(code)", limit = "1")
-        Object serialize(Object code,
-                @Cached RubyStringLibrary strings,
-                @Cached TruffleString.CopyToByteArrayNode copyToByteArrayNode,
-                @Cached TruffleString.FromByteArrayNode fromByteArrayNode) {
-            var tstring = strings.getTString(code);
-            var tencoding = strings.getTEncoding(code);
-            var source = copyToByteArrayNode.execute(tstring, tencoding);
-
-            byte[] serialized = Parser.parseAndSerialize(source);
-
-            return createString(fromByteArrayNode, serialized, Encodings.BINARY);
-        }
-    }
-
-    @CoreMethod(names = { "yarp_parse" }, onSingleton = true, required = 1)
-    public abstract static class YARPParseNode extends CoreMethodArrayArgumentsNode {
-        @TruffleBoundary
-        @Specialization(guards = "strings.isRubyString(code)", limit = "1")
-        Object parse(Object code,
-                @Cached RubyStringLibrary strings,
-                @Cached TruffleString.CopyToByteArrayNode copyToByteArrayNode,
-                @Cached TruffleString.FromJavaStringNode fromJavaStringNode) {
-            var tstring = strings.getTString(code);
-            var tencoding = strings.getTEncoding(code);
-            var source = copyToByteArrayNode.execute(tstring, tencoding);
-
-            byte[] serialized = Parser.parseAndSerialize(source);
-
-            var yarpSource = YARPTranslatorDriver.createYARPSource(source);
-            var parseResult = Loader.load(serialized, yarpSource);
-            var ast = parseResult.value;
-
-            return createString(fromJavaStringNode, ast.toString(), Encodings.UTF_8);
-        }
-    }
-
-    @CoreMethod(names = "yarp_execute", onSingleton = true, required = 1)
-    public abstract static class YARPExecuteNode extends CoreMethodArrayArgumentsNode {
-        @Specialization(guards = "strings.isRubyString(code)", limit = "1")
-        Object yarpExecute(VirtualFrame frame, Object code,
-                @Cached RubyStringLibrary strings) {
-            return doExecute(code, RubyArguments.getMethod(frame));
-        }
-
-        @TruffleBoundary
-        private Object doExecute(Object code, InternalMethod method) {
-            TranslatorEnvironment.resetTemporaryVariablesIndex();
-
-            final RootCallTarget callTarget = getContext().getCodeLoader().parseWithYARP(
-                    code,
-                    ParserContext.TOP_LEVEL,
-                    null,
-                    getContext().getRootLexicalScope(),
-                    null);
-
-            RubyRootNode truffleAST = RubyRootNode.of(callTarget);
-
-            System.err.println("Truffle AST:");
-            NodeUtil.printCompactTree(getContext().getEnvErrStream(), truffleAST);
-
-            return truffleAST.getCallTarget().call(RubyArguments.pack(
-                    null,
-                    null,
-                    method,
-                    DeclarationContext.topLevel(getContext()),
-                    null,
-                    coreLibrary().mainObject,
-                    Nil.INSTANCE,
-                    NoKeywordArgumentsDescriptor.INSTANCE,
-                    EMPTY_ARGUMENTS));
-        }
-    }
-
     @CoreMethod(names = "parse_ast", onSingleton = true, required = 1)
     public abstract static class ParseASTNode extends CoreMethodArrayArgumentsNode {
         @TruffleBoundary
@@ -346,8 +265,9 @@ Object ast(Object code,
 
             var parseResult = YARPTranslatorDriver.parseToYARPAST(getLanguage(), rubySource, Collections.emptyList(),
                     parseEnvironment);
+            var ast = parseResult.value;
 
-            return createString(fromJavaStringNode, parseResult.value.toString(), Encodings.UTF_8);
+            return createString(fromJavaStringNode, ast.toString(), Encodings.UTF_8);
         }
     }