YJIT: Avoid unnecessary BOP invalidation

Previously we would invalidate BOPs in YJIT when the method registered as a BOP was redefined on a subclass.
ruby · Dec 14, 2021 · b7ae089 · b7ae089
1 parent b563f12
commit b7ae089
Showing 1 changed file with 5 additions and 3 deletions.
diff --git a/vm.c b/vm.c
@@ -1885,9 +1885,11 @@ rb_vm_check_redefinition_opt_method(const rb_method_entry_t *me, VALUE klass)
     if (vm_redefinition_check_method_type(me->def)) {
         if (st_lookup(vm_opt_method_def_table, (st_data_t)me->def, &bop)) {
             int flag = vm_redefinition_check_flag(klass);
-            rb_yjit_bop_redefined(klass, me, (enum ruby_basic_operators)bop);
-	    ruby_vm_redefined_flag[bop] |= flag;
-	}
+            if (flag != 0) {
+                rb_yjit_bop_redefined(klass, me, (enum ruby_basic_operators)bop);
+                ruby_vm_redefined_flag[bop] |= flag;
+            }
+        }
     }
 }