Add back test properly

Author: Fidget-Spinner

Lib/test/test_capi/test_opt.py

@@ -5189,6 +5189,9 @@ def testfunc(n):
         self.assertIn("_CHECK_FUNCTION_VERSION", uops)
 
         global_identity_code_will_be_modified.__code__ = (lambda a: 0xdeadead).__code__
+        _testinternalcapi.clear_executor_deletion_list()
+        ex = get_first_executor(testfunc)
+        self.assertIsNone(ex)
         # JItted code should've deopted.
         self.assertEqual(global_identity_code_will_be_modified(None), 0xdeadead)
 

Objects/funcobject.c

@@ -64,6 +64,13 @@ handle_func_event(PyFunction_WatchEvent event, PyFunctionObject *func,
         case PyFunction_EVENT_MODIFY_DEFAULTS:
         case PyFunction_EVENT_MODIFY_KWDEFAULTS:
         case PyFunction_EVENT_MODIFY_QUALNAME:
+#if _Py_TIER2
+            // Note: we only invalidate JIT code if a function version changes.
+            // Not when the function is deallocated.
+            // Function deallocation occurs frequently (think: lambdas),
+            // so we want to minimize dependency invalidation there.
+            _Py_Executors_InvalidateDependency(interp, func, 1);
+#endif
             RARE_EVENT_INTERP_INC(interp, func_modification);
             break;
         default:

Python/optimizer_bytecodes.c

@@ -1009,10 +1009,7 @@ dummy_func(void) {
         }
         // Guarded on this, so it can be promoted.
         sym_set_const(callable, func);
-        // We do not need to add func to the bloom filter, as we never remove
-        // this guard. Note: we generally do not want to add functions to our dependencies,
-        // as we want to avoid having to invalidate all executors on every function
-        // deallocation, which is a common procedure (e.g. lambdas).
+        _Py_BloomFilter_Add(dependencies, func);
     }
 
     op(_CHECK_METHOD_VERSION, (func_version/2, callable, null, unused[oparg] -- callable, null, unused[oparg])) {