[GR-17457] Use a faster frozen check for uncached cases

PullRequest: truffleruby/3292

Author: eregon

src/main/java/org/truffleruby/core/module/ModuleFields.java

@@ -49,7 +49,6 @@
 import org.truffleruby.language.constants.ConstantEntry;
 import org.truffleruby.language.constants.GetConstantNode;
 import org.truffleruby.language.control.RaiseException;
-import org.truffleruby.language.library.RubyLibrary;
 import org.truffleruby.language.library.RubyStringLibrary;
 import org.truffleruby.language.loader.ReentrantLockFreeingMap;
 import org.truffleruby.language.methods.InternalMethod;
@@ -245,7 +244,7 @@ public void initCopy(RubyModule from) {
     }
 
     public void checkFrozen(RubyContext context, Node currentNode) {
-        if (context.getCoreLibrary() != null && RubyLibrary.getUncached().isFrozen(rubyModule)) {
+        if (context.getCoreLibrary() != null && rubyModule.isFrozenUncached()) {
             String name;
             Object receiver = rubyModule;
             if (rubyModule instanceof RubyClass) {

src/main/java/org/truffleruby/language/RubyDynamicObject.java

@@ -9,6 +9,7 @@
  */
 package org.truffleruby.language;
 
+import com.oracle.truffle.api.CompilerAsserts;
 import com.oracle.truffle.api.interop.StopIterationException;
 import com.oracle.truffle.api.interop.UnknownKeyException;
 import org.truffleruby.RubyContext;
@@ -20,6 +21,8 @@
 import org.truffleruby.core.cast.ToLongNode;
 import org.truffleruby.core.kernel.KernelNodes;
 import org.truffleruby.core.klass.RubyClass;
+import org.truffleruby.core.range.RubyRange;
+import org.truffleruby.core.string.RubyString;
 import org.truffleruby.core.string.StringUtils;
 import org.truffleruby.interop.ForeignToRubyArgumentsNode;
 import org.truffleruby.interop.ForeignToRubyNode;
@@ -114,6 +117,12 @@ public boolean isFrozen(
             @CachedLibrary("this") DynamicObjectLibrary objLib) {
         return (objLib.getShapeFlags(this) & FROZEN) != 0;
     }
+
+    public boolean isFrozenUncached() {
+        CompilerAsserts.neverPartOfCompilation("Use RubyLibrary instead in PE code");
+        assert !(this instanceof RubyString) && !(this instanceof RubyRange);
+        return (getShape().getFlags() & FROZEN) != 0;
+    }
     // endregion
 
     // region InteropLibrary messages