Make the CoreLibrary SourceSection static

Author: eregon

src/main/java/org/truffleruby/builtins/CoreMethodNodeManager.java

@@ -13,10 +13,10 @@
 import java.util.function.Function;
 
 import com.oracle.truffle.api.CompilerDirectives;
-import com.oracle.truffle.api.source.SourceSection;
 import org.truffleruby.RubyContext;
 import org.truffleruby.RubyLanguage;
 import org.truffleruby.collections.CachedSupplier;
+import org.truffleruby.core.CoreLibrary;
 import org.truffleruby.core.array.ArrayUtils;
 import org.truffleruby.core.cast.TaintResultNode;
 import org.truffleruby.core.klass.RubyClass;
@@ -208,7 +208,7 @@ private static void addMethod(
         final LexicalScope lexicalScope = new LexicalScope(context.getRootLexicalScope(), module);
 
         for (String name : names) {
-            final SharedMethodInfo sharedMethodInfo = makeSharedMethodInfo(context, lexicalScope, module, name, arity);
+            final SharedMethodInfo sharedMethodInfo = makeSharedMethodInfo(lexicalScope, module, name, arity);
 
             module.fields.addMethod(context, null, new InternalMethod(
                     context,
@@ -223,10 +223,9 @@ private static void addMethod(
         }
     }
 
-    private static SharedMethodInfo makeSharedMethodInfo(RubyContext context, LexicalScope lexicalScope,
-            RubyModule module, String name, Arity arity) {
-        final SourceSection sourceSection = context.getCoreLibrary().sourceSection;
-        return new SharedMethodInfo(sourceSection, lexicalScope, arity, module, name, 0, "builtin", null);
+    private static SharedMethodInfo makeSharedMethodInfo(LexicalScope lexicalScope, RubyModule module, String name,
+            Arity arity) {
+        return new SharedMethodInfo(CoreLibrary.SOURCE_SECTION, lexicalScope, arity, module, name, 0, "builtin", null);
     }
 
     private static Arity createArity(int required, int optional, boolean rest, String keywordAsOptional) {

src/main/java/org/truffleruby/core/CoreLibrary.java

@@ -125,7 +125,7 @@ public class CoreLibrary {
     private final RubyContext context;
     private final RubyLanguage language;
 
-    public final SourceSection sourceSection = initCoreSourceSection();
+    public static final SourceSection SOURCE_SECTION = initCoreSourceSection();
 
     public final RubyClass argumentErrorClass;
     public final RubyClass arrayClass;
@@ -263,7 +263,7 @@ public class CoreLibrary {
     public final String corePath;
 
     @TruffleBoundary
-    private SourceSection initCoreSourceSection() {
+    private static SourceSection initCoreSourceSection() {
         final Source.SourceBuilder builder = Source.newBuilder(TruffleRuby.LANGUAGE_ID, "", "(core)");
         builder.internal(true);
 

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

@@ -45,6 +45,7 @@
 import org.truffleruby.RubyLanguage;
 import org.truffleruby.aot.ParserCache;
 import org.truffleruby.collections.Memo;
+import org.truffleruby.core.CoreLibrary;
 import org.truffleruby.core.kernel.AutoSplitNode;
 import org.truffleruby.core.kernel.ChompLoopNode;
 import org.truffleruby.core.kernel.KernelGetsNode;
@@ -438,7 +439,7 @@ private TranslatorEnvironment environmentForFrame(RubyContext context, Materiali
             return null;
         } else {
             final SharedMethodInfo sharedMethodInfo = new SharedMethodInfo(
-                    context.getCoreLibrary().sourceSection,
+                    CoreLibrary.SOURCE_SECTION,
                     context.getRootLexicalScope(),
                     Arity.NO_ARGUMENTS,
                     null,