requested changes

Author: tom-pytel

Doc/library/tracemalloc.rst

@@ -331,10 +331,10 @@ Functions
    :mod:`tracemalloc` module as a tuple: ``(current: int, peak: int)``.
 
 
-.. function:: get_traced_refs()
+.. function:: get_traced_allocs()
 
-   Get the current count of created and destroyed memory blocks.
-   :mod:`tracemalloc` module as a tuple: ``(created: int, destroyed: int)``.
+   Get the current unit counts of allocated and deallocated memory blocks.
+   :mod:`tracemalloc` module as a tuple: ``(allocs: int, deallocs: int)``.
 
 
 .. function:: reset_peak()

Include/internal/pycore_tracemalloc.h

@@ -94,12 +94,12 @@ struct _tracemalloc_runtime_state {
     /* domain (unsigned int) => traces (_Py_hashtable_t).
        Protected by TABLES_LOCK(). */
     _Py_hashtable_t *domains;
-    /* Number of references created.
+    /* Number of allocations.
        Protected by TABLES_LOCK(). */
-    size_t refs_created;
-    /* Number of references destroyed.
+    Py_ssize_t allocations;
+    /* Number of deallocations.
        Protected by TABLES_LOCK() and sometimes modified atomically. */
-    size_t refs_destroyed;
+    Py_ssize_t deallocations;
 
     struct tracemalloc_traceback empty_traceback;
 
@@ -162,7 +162,7 @@ extern size_t _PyTraceMalloc_GetMemory(void);
 extern PyObject* _PyTraceMalloc_GetTracedMemory(void);
 
 /* Get the current number of references created and destroyed as a 2-tuple */
-extern PyObject* _PyTraceMalloc_GetTracedRefs(void);
+extern PyObject* _PyTraceMalloc_GetTracedAllocs(void);
 
 /* Set the peak size of traced memory blocks to the current size */
 extern void _PyTraceMalloc_ResetPeak(void);

Lib/test/test_tracemalloc.py

@@ -1156,15 +1156,15 @@ def g():
         try:
             tracemalloc.clear_traces()
             f()
-            refs = tracemalloc.get_traced_refs()
+            refs = tracemalloc.get_traced_allocs()
             if refs == (1, 0):
                 warnings.warn("ceval Py_DECREF doesn't emit PyRefTracer_DESTROY in this build")
             else:
                 self.assertEqual(refs, (1, 1))
 
             tracemalloc.clear_traces()
             g()
-            refs = tracemalloc.get_traced_refs()
+            refs = tracemalloc.get_traced_allocs()
             if refs == (3, 2):
                 warnings.warn("ceval Py_DECREF doesn't emit PyRefTracer_DESTROY in this build")
             else:

Modules/_tracemalloc.c

@@ -169,18 +169,18 @@ _tracemalloc_get_traced_memory_impl(PyObject *module)
 
 
 /*[clinic input]
-_tracemalloc.get_traced_refs
+_tracemalloc.get_traced_allocs
 
 Get the current count of created and destroyed refs.
 
 Returns a tuple: (created: int, destroyed: int).
 [clinic start generated code]*/
 
 static PyObject *
-_tracemalloc_get_traced_refs_impl(PyObject *module)
-/*[clinic end generated code: output=81d36fdeb3ffc362 input=d0652f2592733b0e]*/
+_tracemalloc_get_traced_allocs_impl(PyObject *module)
+/*[clinic end generated code: output=cb84cbd781fab85f input=336f38d37a5830ef]*/
 {
-    return _PyTraceMalloc_GetTracedRefs();
+    return _PyTraceMalloc_GetTracedAllocs();
 }
 
 
@@ -212,7 +212,7 @@ static PyMethodDef module_methods[] = {
     _TRACEMALLOC_GET_TRACEBACK_LIMIT_METHODDEF
     _TRACEMALLOC_GET_TRACEMALLOC_MEMORY_METHODDEF
     _TRACEMALLOC_GET_TRACED_MEMORY_METHODDEF
-    _TRACEMALLOC_GET_TRACED_REFS_METHODDEF
+    _TRACEMALLOC_GET_TRACED_ALLOCS_METHODDEF
     _TRACEMALLOC_RESET_PEAK_METHODDEF
     /* sentinel */
     {NULL, NULL}

Modules/clinic/_tracemalloc.c.h

@@ -76,8 +76,8 @@ typedef struct {
 #define tracemalloc_tracebacks _PyRuntime.tracemalloc.tracebacks
 #define tracemalloc_traces _PyRuntime.tracemalloc.traces
 #define tracemalloc_domains _PyRuntime.tracemalloc.domains
-#define tracemalloc_refs_created _PyRuntime.tracemalloc.refs_created
-#define tracemalloc_refs_destroyed _PyRuntime.tracemalloc.refs_destroyed
+#define tracemalloc_allocations _PyRuntime.tracemalloc.allocations
+#define tracemalloc_deallocations _PyRuntime.tracemalloc.deallocations
 
 
 #ifdef TRACE_DEBUG
@@ -1261,8 +1261,8 @@ _PyTraceMalloc_TraceRef(PyObject *op, PyRefTracerEvent event,
 {
     if (event != PyRefTracer_CREATE) {
         /* we don't want bother here with the lock for performance reasons */
-        if (_Py_atomic_load_int32_relaxed(&tracemalloc_config.tracing)) {
-            _Py_atomic_add_ssize((Py_ssize_t *)&tracemalloc_refs_destroyed, 1);
+        if (_Py_atomic_load_int_relaxed(&tracemalloc_config.tracing)) {
+            _Py_atomic_add_ssize(&tracemalloc_deallocations, 1);
         }
         return 0;
     }
@@ -1277,7 +1277,7 @@ _PyTraceMalloc_TraceRef(PyObject *op, PyRefTracerEvent event,
         goto done;
     }
 
-    tracemalloc_refs_created += 1;
+    tracemalloc_allocations += 1;
 
     PyTypeObject *type = Py_TYPE(op);
     const size_t presize = _PyType_PreHeaderSize(type);
@@ -1334,8 +1334,8 @@ _PyTraceMalloc_ClearTraces(void)
     TABLES_LOCK();
     if (tracemalloc_config.tracing) {
         tracemalloc_clear_traces_unlocked();
-        tracemalloc_refs_created = 0;
-        tracemalloc_refs_destroyed = 0;
+        tracemalloc_allocations = 0;
+        tracemalloc_deallocations = 0;
     }
     TABLES_UNLOCK();
 }
@@ -1476,21 +1476,21 @@ _PyTraceMalloc_GetTracedMemory(void)
 
 
 PyObject *
-_PyTraceMalloc_GetTracedRefs(void)
+_PyTraceMalloc_GetTracedAllocs(void)
 {
     TABLES_LOCK();
-    Py_ssize_t created, destroyed;
+    Py_ssize_t allocations, deallocations;
     if (tracemalloc_config.tracing) {
-        created = tracemalloc_refs_created;
-        destroyed = tracemalloc_refs_destroyed;
+        allocations = tracemalloc_allocations;
+        deallocations = tracemalloc_deallocations;
     }
     else {
-        created = 0;
-        destroyed = 0;
+        allocations = 0;
+        deallocations = 0;
     }
     TABLES_UNLOCK();
 
-    return Py_BuildValue("nn", created, destroyed);
+    return Py_BuildValue("nn", allocations, deallocations);
 }
 
 void