Merge pull request #205 from Shopify/stats_no_late_enable

maximecb · web-flow · commit 22a604f3aeb7 · 2021-09-07T15:11:27.000-07:00
Prevent stats being enabled late at run-time
diff --git a/yjit.rb b/yjit.rb
@@ -138,10 +138,6 @@ def self.stats_enabled?
     Primitive.cexpr! 'rb_yjit_opts.gen_stats ? Qtrue : Qfalse'
   end
 
-  def self.stats_enabled=(enabled)
-    Primitive.set_stats_enabled(enabled)
-  end
-
   def self.enabled?
     Primitive.cexpr! 'rb_yjit_enabled_p() ? Qtrue : Qfalse'
   end
diff --git a/yjit_iface.c b/yjit_iface.c
@@ -811,13 +811,6 @@ reset_stats_bang(rb_execution_context_t *ec, VALUE self)
     return Qnil;
 }
 
-static VALUE
-set_stats_enabled(rb_execution_context_t *ec, VALUE self, VALUE enabled)
-{
-    rb_yjit_opts.gen_stats = RB_TEST(enabled);
-    return enabled;
-}
-
 #include "yjit.rbinc"
 
 #if YJIT_STATS
