New --jit options

Author: chrisseaton

doc/user/compatibility.md

@@ -120,6 +120,9 @@ Programs passed in `-e` arguments with magic-comments must have an encoding that
 is UTF-8 or a subset of UTF-8, as the JVM has already decoded arguments by the
 time we get them.
 
+`--jit` options and the `jit` feature are not supported because TruffleRuby
+uses Graal as a JIT.
+
 #### Setting the process title doesn't always work
 
 Setting the process title (via `$0` or `Process.setproctitle` in Ruby) is done

spec/tags/truffle/launcher_tags.txt

@@ -59,3 +59,7 @@ slow:The launcher sets the log level using --log.ruby.level=
 fails(GR-13956):The launcher prints help containing runtime options
 slow:The launcher logs options if --options.log is set
 slow:The launcher sets the log level using --log.level=
+slow:The launcher ignores --jit option with a warning
+slow:The launcher ignores --jit-... options with a warning and a hint to look at Graal options
+slow:The launcher ignores --jit... options with a warning and a hint to look at Graal documentation
+slow:The launcher warns on ignored options

spec/truffle/launcher_spec.rb

@@ -300,4 +300,37 @@ def should_print_full_java_command(options, env: {})
       out.should include("[truffle] opt done")
     end
   end
+
+  it "ignores --jit... options with a warning and a hint to look at Graal documentation" do
+    [
+      "--jit",
+      "--jit-warnings",
+      "--jit-debug",
+      "--jit-wait",
+      "--jit-save-temps",
+      "--jit-verbose",
+      "--jit-max-cache",
+      "--jit-min-calls",
+    ].each do |option|
+      out = ruby_exe("p 14", options: option, args: "2>&1")
+      $?.success?.should == true
+      out.should include("JIT options are not supported - see the Graal documentation instead")
+      out.should include("14")
+    end
+  end
+  
+  it "warns on ignored options" do
+    [
+      "-y",
+      "--yydebug",
+      "--debug-frozen-string-literal",
+      "--dump=insns",
+    ].each do |option|
+      out = ruby_exe("p 14", options: option, args: "2>&1")
+      $?.success?.should == true
+      out.should include("[ruby] WARNING the #{option} switch is silently ignored as it is an internal development tool")
+      out.should include("14")
+    end
+  end
+
 end

src/launcher/java/org/truffleruby/launcher/RubyLauncher.java

@@ -163,6 +163,14 @@ protected void collectArguments(Set<String> options) {
                 "--external-encoding", "--internal-encoding",
                 "--version",
                 "--help",
+                "--jit",
+                "--jit-warnings",
+                "--jit-debug",
+                "--jit-wait",
+                "--jit-save-temps",
+                "--jit-verbose",
+                "--jit-max-cache",
+                "--jit-min-calls"));
     }
 
     @Override

src/launcher/java/org/truffleruby/launcher/options/CommandLineParser.java

@@ -464,6 +464,8 @@ private void processArgument() throws CommandLineException {
                         break FOR;
                     } else if (argument.startsWith("--dump=")) {
                         LOGGER.warning("the --dump= switch is silently ignored as it is an internal development tool");
+                    } else if (argument.equals("--jit") || argument.startsWith("--jit-")) {
+                        LOGGER.warning("JIT options are not supported - see the Graal documentation instead");
                         break FOR;
                     } else {
                         if (argument.equals("--")) {