Restore a test, address review

Author: Fidget-Spinner

Include/internal/pycore_interp_structs.h

@@ -755,7 +755,6 @@ struct _Py_unique_id_pool {
 
 typedef _Py_CODEUNIT *(*_PyJitEntryFuncPtr)(struct _PyExecutorObject *exec, _PyInterpreterFrame *frame, _PyStackRef *stack_pointer, PyThreadState *tstate);
 
-
 /* PyInterpreterState holds the global state for one of the runtime's
    interpreters.  Typically the initial (main) interpreter is the only one.
 

Lib/test/test_capi/test_opt.py

@@ -139,7 +139,6 @@ def testfunc(x):
         self.assertIn("_JUMP_TO_TOP", uops)
         self.assertIn("_LOAD_FAST_BORROW_0", uops)
 
-    @unittest.skip("gh-139109 WIP")
     def test_extended_arg(self):
         "Check EXTENDED_ARG handling in superblock creation"
         ns = {}

Python/optimizer.c

@@ -125,10 +125,6 @@ _PyOptimizer_Optimize(
     assert(!interp->compiling);
     assert(_tstate->jit_tracer_state.initial_state.stack_depth >= 0);
 #ifndef Py_GIL_DISABLED
-    // Trace got stomped on by another thread.
-    if (_tstate->jit_tracer_state.initial_state.func == NULL) {
-        return 0;
-    }
     assert(_tstate->jit_tracer_state.initial_state.func != NULL);
     interp->compiling = true;
     // The first executor in a chain and the MAX_CHAIN_DEPTH'th executor *must*