revert #16672 for incorrect semantics

diff --git a/Misc/NEWS b/Misc/NEWS
index b905afbed8dbc002a6638b7d1ef8c46398f05d7e..5dc5466dc439a0718926b7a29ed47f5bdba1357f 100644 (file)
--- a/Misc/NEWS
+++ b/Misc/NEWS
@@ -50,8 +50,6 @@ Core and Builtins
 - Issue #15422: Get rid of PyCFunction_New macro. Use PyCFunction_NewEx
   function (PyCFunction_New func is still present for backward compatibility).
 
-- Issue #16672: Improve performance tracing performance
-
 - Issue #14470: Remove w9xpopen support per PEP 11.
 
 - Issue #9856: Replace deprecation warning with raising TypeError

diff --git a/Python/ceval.c b/Python/ceval.c
index 5f6f35cb54f7c105b6032767095a529d73e88f85..32c203ecbcf030aaa73d5dd16de4c6eafe668404 100644 (file)
--- a/Python/ceval.c
+++ b/Python/ceval.c
@@ -1277,8 +1277,7 @@ PyEval_EvalFrameEx(PyFrameObject *f, int throwflag)
         /* line-by-line tracing support */
 
         if (_Py_TracingPossible &&
-                tstate->c_tracefunc != NULL && !tstate->tracing &&
-                f->f_trace != NULL) {
+            tstate->c_tracefunc != NULL && !tstate->tracing) {
             int err;
             /* see maybe_call_line_trace
                for expository comments */
@@ -3009,7 +3008,7 @@ error:
         /* Log traceback info. */
         PyTraceBack_Here(f);
 
-        if (tstate->c_tracefunc != NULL && f->f_trace != NULL)
+        if (tstate->c_tracefunc != NULL)
             call_exc_trace(tstate->c_tracefunc, tstate->c_traceobj, f);
 
 fast_block_end:
@@ -3128,7 +3127,7 @@ fast_yield:
     }
 
     if (tstate->use_tracing) {
-        if (tstate->c_tracefunc && f->f_trace != NULL) {
+        if (tstate->c_tracefunc) {
             if (why == WHY_RETURN || why == WHY_YIELD) {
                 if (call_trace(tstate->c_tracefunc,
                                tstate->c_traceobj, f,