[GR-17457] Pass benchmark flags in the order they are printed for clarity

PullRequest: truffleruby/2396

Author: eregon

bench/benchmark-interface/README.md

@@ -164,9 +164,8 @@ loop printing the iterations per second as it goes. By default it runs for
 10s, printing the time every second, but you can use options like `--time 60`
 to run for 60 seconds, and `--freq 0.1` to print ten times a second.
 
-You can also print the elapsed time since the benchmark started with the
-`--elapsed` flag, and the actual number of iterations with the `--iterations`
-flag.
+You can also print the actual number of iterations with the `--iterations` flag
+and the elapsed time since the benchmark started with the `--elapsed` flag.
 
 #### `bm` and `bmbm`
 

bench/rails/blog6-bundle-install.rb

@@ -1,8 +1,7 @@
 # This is a long running benchmark and should
 # be run using fixed iterations. E.g.
-# jt benchmark bench/rails/blog6-bundle-install.rb --elapsed --iterations --ips --fixed-iterations 1
-# RUBY_BENCHMARKS=true jt mx benchmark ecosystem
-#
+# jt benchmark bench/rails/blog6-bundle-install.rb --iterations --elapsed --ips --fixed-iterations 1
+# RUBY_BENCHMARKS=true jt mx benchmark ruby-warmup:rails/blog6-bundle-install
 
 require_relative './blog6-setup.rb'
 

bench/rails/blog6-rails-routes.rb

@@ -1,8 +1,7 @@
 # This is a long running benchmark and should
 # be run using fixed iterations. E.g.
-# jt benchmark bench/rails/blog6-rails-routes.rb --elapsed --iterations --ips --fixed-iterations 1
-# RUBY_BENCHMARKS=true jt mx benchmark ecosystem
-#
+# jt benchmark bench/rails/blog6-rails-routes.rb --iterations --elapsed --ips --fixed-iterations 1
+# RUBY_BENCHMARKS=true jt mx benchmark ruby-warmup:rails/blog6-rails-routes
 
 require_relative './blog6-setup.rb'
 require_relative '../../tool/jt.rb'
@@ -12,6 +11,6 @@
 
 benchmark 'blog6-rails-routes' do
   Dir.chdir(Blog6Setup::BLOG6_DIR) do
-    JT.ruby(*%w[-S bin/rails routes])
+    JT.ruby(*%w[bin/rails routes])
   end
 end

mx.truffleruby/mx_truffleruby_benchmark.py

@@ -281,7 +281,7 @@ def runBenchmark(self, benchmark, bmSuiteArgs):
 
         arguments = ['benchmark']
         if self.config()['kind'] == 'simple':
-            arguments.extend(['--simple', '--elapsed', '--iterations'])
+            arguments.extend(['--simple', '--iterations', '--elapsed'])
             time = self.time()
             if isinstance(time, dict):
                 if benchmark in time:
@@ -295,7 +295,7 @@ def runBenchmark(self, benchmark, bmSuiteArgs):
             iterations_config = self.config()['iterations'][benchmark]
             fixed_iterations = sorted(iterations_config.keys())
             fixed_iterations_arg = ','.join([str(i) for i in fixed_iterations])
-            arguments.extend(['--elapsed', '--iterations', '--ips'])
+            arguments.extend(['--iterations', '--elapsed', '--ips'])
             arguments.extend(['--fixed-iterations'])
             arguments.extend([fixed_iterations_arg])
             if iterations_config != {1:'single-shot'}: