Remove --prism option

Author: andrykonchin

src/main/java/org/truffleruby/options/LanguageOptions.java

@@ -131,8 +131,6 @@ public final class LanguageOptions {
     public final boolean RUN_TWICE;
     /** --experimental-engine-caching=RUN_TWICE */
     public final boolean EXPERIMENTAL_ENGINE_CACHING;
-    /** --prism=true */
-    public final boolean PRISM;
 
     public LanguageOptions(Env env, OptionValues options, boolean singleContext) {
         CORE_LOAD_PATH = options.get(OptionsCatalog.CORE_LOAD_PATH_KEY);
@@ -189,7 +187,6 @@ public LanguageOptions(Env env, OptionValues options, boolean singleContext) {
         SHARED_OBJECTS_FORCE = options.get(OptionsCatalog.SHARED_OBJECTS_FORCE_KEY);
         RUN_TWICE = options.get(OptionsCatalog.RUN_TWICE_KEY);
         EXPERIMENTAL_ENGINE_CACHING = options.hasBeenSet(OptionsCatalog.EXPERIMENTAL_ENGINE_CACHING_KEY) ? options.get(OptionsCatalog.EXPERIMENTAL_ENGINE_CACHING_KEY) : RUN_TWICE;
-        PRISM = options.get(OptionsCatalog.PRISM_KEY);
     }
 
     public Object fromDescriptor(OptionDescriptor descriptor) {
@@ -302,8 +299,6 @@ public Object fromDescriptor(OptionDescriptor descriptor) {
                 return RUN_TWICE;
             case "ruby.experimental-engine-caching":
                 return EXPERIMENTAL_ENGINE_CACHING;
-            case "ruby.prism":
-                return PRISM;
             default:
                 return null;
         }
@@ -363,8 +358,7 @@ public static boolean areOptionsCompatible(OptionValues one, OptionValues two) {
                one.get(OptionsCatalog.SHARED_OBJECTS_DEBUG_KEY).equals(two.get(OptionsCatalog.SHARED_OBJECTS_DEBUG_KEY)) &&
                one.get(OptionsCatalog.SHARED_OBJECTS_FORCE_KEY).equals(two.get(OptionsCatalog.SHARED_OBJECTS_FORCE_KEY)) &&
                one.get(OptionsCatalog.RUN_TWICE_KEY).equals(two.get(OptionsCatalog.RUN_TWICE_KEY)) &&
-               one.get(OptionsCatalog.EXPERIMENTAL_ENGINE_CACHING_KEY).equals(two.get(OptionsCatalog.EXPERIMENTAL_ENGINE_CACHING_KEY)) &&
-               one.get(OptionsCatalog.PRISM_KEY).equals(two.get(OptionsCatalog.PRISM_KEY));
+               one.get(OptionsCatalog.EXPERIMENTAL_ENGINE_CACHING_KEY).equals(two.get(OptionsCatalog.EXPERIMENTAL_ENGINE_CACHING_KEY));
     }
 
     public static boolean areOptionsCompatibleOrLog(TruffleLogger logger, LanguageOptions oldOptions, LanguageOptions newOptions) {
@@ -749,13 +743,6 @@ public static boolean areOptionsCompatibleOrLog(TruffleLogger logger, LanguageOp
             return false;
         }
 
-        oldValue = oldOptions.PRISM;
-        newValue = newOptions.PRISM;
-        if (!newValue.equals(oldValue)) {
-            logger.fine("not reusing pre-initialized context: --prism differs, was: " + oldValue + " and is now: " + newValue);
-            return false;
-        }
-
         return true;
     }
 }

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

@@ -106,7 +106,7 @@ public RootCallTarget parse(RubySource rubySource, ParserContext parserContext,
             MaterializedFrame parentFrame, LexicalScope staticLexicalScope, Node currentNode) {
         this.parseEnvironment = new ParseEnvironment(language, rubySource, null, parserContext, currentNode);
 
-        if (language.options.PRISM) {
+        if (true) {
             return new YARPTranslatorDriver(context).parse(rubySource, parserContext, argumentNames,
                     parentFrame, staticLexicalScope, currentNode);
         }

src/options.yml

@@ -52,7 +52,6 @@ LANGUAGE_OPTIONS:
 - RUN_TWICE
 - CHECK_CLONE_UNINITIALIZED_CORRECTNESS
 - ALWAYS_CLONE_ALL
-- PRISM
 - BUILDING_CORE_CEXTS
 - INSTRUMENT_ALL_NODES
 
@@ -264,5 +263,3 @@ INTERNAL: # Options for debugging the TruffleRuby implementation
 
     # Options for the regular expression engines
     COMPARE_REGEX_ENGINES: [compare-regex-engines, boolean, false, 'Uses both Joni and the TRegex engine and compares their results']
-
-    PRISM: [prism, boolean, true, 'Use Prism as the parser for Ruby code']

src/shared/java/org/truffleruby/shared/options/OptionsCatalog.java

@@ -164,7 +164,6 @@ public final class OptionsCatalog {
     public static final OptionKey<Boolean> RUN_TWICE_KEY = new OptionKey<>(false);
     public static final OptionKey<Boolean> EXPERIMENTAL_ENGINE_CACHING_KEY = new OptionKey<>(RUN_TWICE_KEY.getDefaultValue());
     public static final OptionKey<Boolean> COMPARE_REGEX_ENGINES_KEY = new OptionKey<>(false);
-    public static final OptionKey<Boolean> PRISM_KEY = new OptionKey<>(true);
 
     public static final OptionDescriptor LOAD_PATHS = OptionDescriptor
             .newBuilder(LOAD_PATHS_KEY, "ruby.load-paths")
@@ -1318,14 +1317,6 @@ public final class OptionsCatalog {
             .usageSyntax("")
             .build();
 
-    public static final OptionDescriptor PRISM = OptionDescriptor
-            .newBuilder(PRISM_KEY, "ruby.prism")
-            .help("Use Prism as the parser for Ruby code")
-            .category(OptionCategory.INTERNAL)
-            .stability(OptionStability.EXPERIMENTAL)
-            .usageSyntax("")
-            .build();
-
     public static OptionDescriptor fromName(String name) {
         switch (name) {
             case "ruby.load-paths":
@@ -1616,8 +1607,6 @@ public static OptionDescriptor fromName(String name) {
                 return EXPERIMENTAL_ENGINE_CACHING;
             case "ruby.compare-regex-engines":
                 return COMPARE_REGEX_ENGINES;
-            case "ruby.prism":
-                return PRISM;
             default:
                 return null;
         }
@@ -1769,7 +1758,6 @@ public static OptionDescriptor[] allDescriptors() {
             RUN_TWICE,
             EXPERIMENTAL_ENGINE_CACHING,
             COMPARE_REGEX_ENGINES,
-            PRISM,
         };
     }
 }