Avoid going through RubyLibrary for RubyString#{isFrozen,freeze}

* Saves footprint and indirections.

Author: eregon

src/main/java/org/truffleruby/core/hash/FreezeHashKeyIfNeededNode.java

@@ -15,10 +15,8 @@
 import org.truffleruby.core.string.ImmutableRubyString;
 import org.truffleruby.language.RubyBaseNode;
 import org.truffleruby.language.dispatch.DispatchNode;
-import org.truffleruby.language.library.RubyLibrary;
 
 import com.oracle.truffle.api.dsl.Specialization;
-import com.oracle.truffle.api.library.CachedLibrary;
 
 @GenerateUncached
 public abstract class FreezeHashKeyIfNeededNode extends RubyBaseNode {
@@ -30,31 +28,21 @@ protected Object immutable(ImmutableRubyString string, boolean compareByIdentity
         return string;
     }
 
-    @Specialization(
-            guards = "rubyLibrary.isFrozen(string)",
-            limit = "getRubyLibraryCacheLimit()")
-    protected Object alreadyFrozen(RubyString string, boolean compareByIdentity,
-            @CachedLibrary("string") RubyLibrary rubyLibrary) {
+    @Specialization(guards = "string.isFrozen()")
+    protected Object alreadyFrozen(RubyString string, boolean compareByIdentity) {
         return string;
     }
 
-    @Specialization(
-            guards = { "!rubyLibrary.isFrozen(string)", "!compareByIdentity" },
-            limit = "getRubyLibraryCacheLimit()")
+    @Specialization(guards = { "!string.isFrozen()", "!compareByIdentity" })
     protected Object dupAndFreeze(RubyString string, boolean compareByIdentity,
-            @CachedLibrary("string") RubyLibrary rubyLibrary,
-            @CachedLibrary(limit = "getRubyLibraryCacheLimit()") RubyLibrary rubyLibraryObject,
             @Cached DispatchNode dupNode) {
-        final Object object = dupNode.call(string, "dup");
-        rubyLibraryObject.freeze(object);
-        return object;
+        final RubyString copy = (RubyString) dupNode.call(string, "dup");
+        copy.freeze();
+        return copy;
     }
 
-    @Specialization(
-            guards = { "!rubyLibrary.isFrozen(string)", "compareByIdentity" },
-            limit = "getRubyLibraryCacheLimit()")
-    protected Object compareByIdentity(RubyString string, boolean compareByIdentity,
-            @CachedLibrary("string") RubyLibrary rubyLibrary) {
+    @Specialization(guards = { "!string.isFrozen()", "compareByIdentity" })
+    protected Object compareByIdentity(RubyString string, boolean compareByIdentity) {
         return string;
     }
 

src/main/java/org/truffleruby/core/string/RubyString.java

@@ -90,12 +90,12 @@ protected String getJavaString() {
 
     // region RubyLibrary messages
     @ExportMessage
-    protected void freeze() {
+    public void freeze() {
         frozen = true;
     }
 
     @ExportMessage
-    protected boolean isFrozen() {
+    public boolean isFrozen() {
         return frozen;
     }
     // endregion