From 1c7effaefdd61f507a0ff803c9aa241d0e65c8c0 Mon Sep 17 00:00:00 2001 From: Randy Stauner Date: Thu, 12 Sep 2024 08:34:33 -0700 Subject: [PATCH] Don't let Rails auto-enable YJIT when benchmarking CRuby baseline --- benchmarks/erubi-rails/config/environments/production.rb | 3 +++ benchmarks/lobsters/config/environments/production.rb | 3 +++ benchmarks/railsbench/config/environments/production.rb | 3 +++ 3 files changed, 9 insertions(+) diff --git a/benchmarks/erubi-rails/config/environments/production.rb b/benchmarks/erubi-rails/config/environments/production.rb index 28cce512..d2ba2e56 100644 --- a/benchmarks/erubi-rails/config/environments/production.rb +++ b/benchmarks/erubi-rails/config/environments/production.rb @@ -92,4 +92,7 @@ # yjit-bench configurations config.active_record.sqlite3_production_warning = false + # If we want to benchmark with YJIT then it has already been enabled by command line arguments. + # If we are benchmarking CRuby without YJIT don't enable it even if this build has it. + config.yjit = false end diff --git a/benchmarks/lobsters/config/environments/production.rb b/benchmarks/lobsters/config/environments/production.rb index 57aeebab..396ca723 100644 --- a/benchmarks/lobsters/config/environments/production.rb +++ b/benchmarks/lobsters/config/environments/production.rb @@ -91,4 +91,7 @@ config.active_record.sqlite3_production_warning = false config.logger = nil config.secret_key_base = 'in general secret should not be in the git repo but this is a benchmark' + # If we want to benchmark with YJIT then it has already been enabled by command line arguments. + # If we are benchmarking CRuby without YJIT don't enable it even if this build has it. + config.yjit = false end diff --git a/benchmarks/railsbench/config/environments/production.rb b/benchmarks/railsbench/config/environments/production.rb index a9301901..6ce7d2f1 100644 --- a/benchmarks/railsbench/config/environments/production.rb +++ b/benchmarks/railsbench/config/environments/production.rb @@ -94,4 +94,7 @@ config.active_record.sqlite3_production_warning = false config.logger = nil config.secret_key_base = 'in general secret should not be in the git repo but this is a benchmark' + # If we want to benchmark with YJIT then it has already been enabled by command line arguments. + # If we are benchmarking CRuby without YJIT don't enable it even if this build has it. + config.yjit = false end