Revert "Update"

This reverts commit a49bcab.

Author: corona10

Include/internal/pycore_interp_structs.h

@@ -899,6 +899,8 @@ struct _is {
     struct _stoptheworld_state stoptheworld;
     struct _qsbr_shared qsbr;
 
+    struct _PyUOpInstruction *jit_uop_buffer;
+
 #if defined(Py_GIL_DISABLED)
     struct _mimalloc_interp_state mimalloc;
     struct _brc_state brc;  // biased reference counting state
@@ -950,8 +952,6 @@ struct _is {
     struct callable_cache callable_cache;
     PyObject *common_consts[NUM_COMMON_CONSTANTS];
     bool jit;
-    bool compiling;
-    struct _PyUOpInstruction *jit_uop_buffer;
     struct _PyExecutorObject *executor_list_head;
     struct _PyExecutorObject *executor_deletion_list_head;
     struct _PyExecutorObject *cold_executor;

Python/bytecodes.c

@@ -2961,9 +2961,7 @@ dummy_func(
                     start--;
                 }
                 _PyExecutorObject *executor;
-                assert(!_PyInterpreterState_GET()->compiling);
                 int optimized = _PyOptimizer_Optimize(frame, start, &executor, 0);
-                assert(!_PyInterpreterState_GET()->compiling);
                 if (optimized <= 0) {
                     this_instr[1].counter = restart_backoff_counter(counter);
                     ERROR_IF(optimized < 0);

Python/generated_cases.c.h

@@ -116,10 +116,7 @@ _PyOptimizer_Optimize(
     _PyExecutorObject **executor_ptr, int chain_depth)
 {
     _PyStackRef *stack_pointer = frame->stackpointer;
-    PyInterpreterState *interp = _PyInterpreterState_GET();
-    assert(interp->jit);
-    assert(!interp->compiling);
-    interp->compiling = true;
+    assert(_PyInterpreterState_GET()->jit);
     // The first executor in a chain and the MAX_CHAIN_DEPTH'th executor *must*
     // make progress in order to avoid infinite loops or excessively-long
     // side-exit chains. We can only insert the executor into the bytecode if
@@ -128,14 +125,12 @@ _PyOptimizer_Optimize(
     bool progress_needed = chain_depth == 0;
     PyCodeObject *code = _PyFrame_GetCode(frame);
     assert(PyCode_Check(code));
-    int ret = 0;
     if (progress_needed && !has_space_for_executor(code, start)) {
-        goto end;
+        return 0;
     }
     int err = uop_optimize(frame, start, executor_ptr, (int)(stack_pointer - _PyFrame_Stackbase(frame)), progress_needed);
     if (err <= 0) {
-        ret = err;
-        goto end;
+        return err;
     }
     assert(*executor_ptr != NULL);
     if (progress_needed) {
@@ -148,7 +143,7 @@ _PyOptimizer_Optimize(
              * it might get confused by the executor disappearing,
              * but there is not much we can do about that here. */
             Py_DECREF(*executor_ptr);
-            goto end;
+            return 0;
         }
         insert_executor(code, start, index, *executor_ptr);
     }
@@ -157,9 +152,7 @@ _PyOptimizer_Optimize(
     }
     (*executor_ptr)->vm_data.chain_depth = chain_depth;
     assert((*executor_ptr)->vm_data.valid);
-end:
-    interp->compiling = false;
-    return ret;
+    return 1;
 }
 
 static _PyExecutorObject *
@@ -1288,7 +1281,6 @@ uop_optimize(
     _PyBloomFilter dependencies;
     _Py_BloomFilter_Init(&dependencies);
     PyInterpreterState *interp = _PyInterpreterState_GET();
-    assert(interp->compiling);
     if (interp->jit_uop_buffer == NULL) {
         interp->jit_uop_buffer = (_PyUOpInstruction *)_PyObject_VirtualAlloc(UOP_BUFFER_SIZE);
         if (interp->jit_uop_buffer == NULL) {

Python/optimizer.c

@@ -577,7 +577,6 @@ init_interpreter(PyInterpreterState *interp,
     }
     interp->_code_object_generation = 0;
     interp->jit = false;
-    interp->compiling = false;
     interp->executor_list_head = NULL;
     interp->executor_deletion_list_head = NULL;
     interp->executor_deletion_list_remaining_capacity = 0;