From 508b0edd561b750c94dfa7f7f3c144aa0076b7c8 Mon Sep 17 00:00:00 2001 From: Matt Valentine-House Date: Thu, 26 Feb 2026 14:44:29 +0000 Subject: [PATCH] Remove GC.compact from the benchmark harness --- harness/harness-common.rb | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/harness/harness-common.rb b/harness/harness-common.rb index cc39b258..05b90a9f 100644 --- a/harness/harness-common.rb +++ b/harness/harness-common.rb @@ -146,9 +146,8 @@ def return_results(warmup_iterations, bench_iterations) yjit_stats = RubyVM::YJIT.runtime_stats if defined?(RubyVM::YJIT.enabled?) && RubyVM::YJIT.enabled? zjit_stats = RubyVM::ZJIT.stats if defined?(RubyVM::ZJIT.enabled?) && RubyVM::ZJIT.enabled? - # Full GC then compact before measuring RSS so fragmentation doesn't inflate the number. + # Full GC before measuring RSS to lower GC variance. GC.start(full_mark: true, immediate_sweep: true) - GC.compact if GC.respond_to?(:compact) rss = get_rss ruby_bench_results["rss"] = rss