Tracing of Python functions is handled at entry not call.

markshannon · markshannon · commit 0721cea88f11 · 2021-09-23T11:02:23.000+01:00
diff --git a/Python/ceval.c b/Python/ceval.c
@@ -4641,7 +4641,7 @@ _PyEval_EvalFrameDefault(PyThreadState *tstate, InterpreterFrame *frame, int thr
             if (Py_TYPE(function) == &PyFunction_Type) {
                 PyCodeObject *code = (PyCodeObject*)PyFunction_GET_CODE(function);
                 int is_coro = code->co_flags & (CO_GENERATOR | CO_COROUTINE | CO_ASYNC_GENERATOR);
-                inline_call = (is_coro || cframe.use_tracing) ? 0 : 1;
+                inline_call = !is_coro;
             }
 
             if (!inline_call) {
@@ -4656,7 +4656,6 @@ _PyEval_EvalFrameDefault(PyThreadState *tstate, InterpreterFrame *frame, int thr
                 DISPATCH();
             }
 
-            assert(!cframe.use_tracing);
             InterpreterFrame *new_frame = _PyEval_FrameFromPyFunctionAndArgs(tstate, stack_pointer-oparg, oparg, function);
             if (new_frame == NULL) {
                 // When we exit here, we own all variables in the stack (the frame creation has not stolen

Original file line number	Diff line number	Diff line change
`@@ -4641,7 +4641,7 @@ _PyEval_EvalFrameDefault(PyThreadState tstate, InterpreterFrame frame, int thr`
`4641`	`4641`	`if (Py_TYPE(function) == &PyFunction_Type) {`
`4642`	`4642`	`PyCodeObject code = (PyCodeObject)PyFunction_GET_CODE(function);`
`4643`	`4643`	`int is_coro = code->co_flags & (CO_GENERATOR \| CO_COROUTINE \| CO_ASYNC_GENERATOR);`
`4644`		`- inline_call = (is_coro \|\| cframe.use_tracing) ? 0 : 1;`
	`4644`	`+ inline_call = !is_coro;`
`4645`	`4645`	`}`
`4646`	`4646`
`4647`	`4647`	`if (!inline_call) {`
`@@ -4656,7 +4656,6 @@ _PyEval_EvalFrameDefault(PyThreadState tstate, InterpreterFrame frame, int thr`
`4656`	`4656`	`DISPATCH();`
`4657`	`4657`	`}`
`4658`	`4658`
`4659`		`- assert(!cframe.use_tracing);`
`4660`	`4659`	`InterpreterFrame *new_frame = _PyEval_FrameFromPyFunctionAndArgs(tstate, stack_pointer-oparg, oparg, function);`
`4661`	`4660`	`if (new_frame == NULL) {`
`4662`	`4661`	`// When we exit here, we own all variables in the stack (the frame creation has not stolen`