Use a dedicated node for frozen String literals, to avoid an extra DefinedWrapperNode

Author: eregon

src/main/java/org/truffleruby/language/literal/FrozenStringLiteralNode.java

@@ -0,0 +1,39 @@
+/*
+ * Copyright (c) 2021 Oracle and/or its affiliates. All rights reserved. This
+ * code is released under a tri EPL/GPL/LGPL license. You can use it,
+ * redistribute it and/or modify it under the terms of the:
+ *
+ * Eclipse Public License version 2.0, or
+ * GNU General Public License version 2, or
+ * GNU Lesser General Public License version 2.1.
+ */
+package org.truffleruby.language.literal;
+
+import org.truffleruby.RubyContext;
+import org.truffleruby.RubyLanguage;
+import org.truffleruby.core.string.CoreString;
+import org.truffleruby.core.string.ImmutableRubyString;
+import org.truffleruby.language.RubyContextSourceNode;
+
+import com.oracle.truffle.api.frame.VirtualFrame;
+
+public class FrozenStringLiteralNode extends RubyContextSourceNode {
+
+    private final ImmutableRubyString frozenString;
+    private final CoreString definition;
+
+    public FrozenStringLiteralNode(ImmutableRubyString frozenString, CoreString definition) {
+        this.frozenString = frozenString;
+        this.definition = definition;
+    }
+
+    @Override
+    public ImmutableRubyString execute(VirtualFrame frame) {
+        return frozenString;
+    }
+
+    @Override
+    public Object isDefined(VirtualFrame frame, RubyLanguage language, RubyContext context) {
+        return definition.createInstance(context);
+    }
+}

src/main/java/org/truffleruby/parser/BodyTranslator.java

@@ -108,6 +108,7 @@
 import org.truffleruby.language.literal.BooleanLiteralNode;
 import org.truffleruby.language.literal.EncodingLiteralNode;
 import org.truffleruby.language.literal.FloatLiteralNode;
+import org.truffleruby.language.literal.FrozenStringLiteralNode;
 import org.truffleruby.language.literal.IntegerFixnumLiteralNode;
 import org.truffleruby.language.literal.LongFixnumLiteralNode;
 import org.truffleruby.language.literal.NilLiteralNode;
@@ -506,13 +507,11 @@ public RubyNode visitCallNode(CallParseNode node) {
         if (receiver instanceof StrParseNode && methodName.equals("freeze")) {
             final StrParseNode strNode = (StrParseNode) receiver;
             final Rope nodeRope = strNode.getValue();
-
             final ImmutableRubyString frozenString = language
                     .getFrozenStringLiteral(nodeRope.getBytes(), nodeRope.getEncoding(), strNode.getCodeRange());
-
             return addNewlineIfNeeded(node, withSourceSection(
                     sourceSection,
-                    new DefinedWrapperNode(language.coreStrings.METHOD, new ObjectLiteralNode(frozenString))));
+                    new FrozenStringLiteralNode(frozenString, language.coreStrings.METHOD)));
         }
 
         if (receiver instanceof ConstParseNode &&
@@ -2862,8 +2861,7 @@ public RubyNode visitStrNode(StrParseNode node) {
         if (node.isFrozen()) {
             final ImmutableRubyString frozenString = language
                     .getFrozenStringLiteral(nodeRope.getBytes(), nodeRope.getEncoding(), node.getCodeRange());
-
-            ret = new DefinedWrapperNode(language.coreStrings.EXPRESSION, new ObjectLiteralNode(frozenString));
+            ret = new FrozenStringLiteralNode(frozenString, language.coreStrings.EXPRESSION);
         } else {
             final LeafRope cachedRope = language.ropeCache
                     .getRope(nodeRope.getBytes(), nodeRope.getEncoding(), node.getCodeRange());