mirrors/cpython
1
0
Fork 0
mirror of https://github.com/python/cpython.git synced 2025-08-03 16:39:00 +00:00
Code Issues Projects Releases Packages Wiki Activity Actions 8

GH-111339: Fix initialization and finalization of static optimizer types (GH-111430)

Browse source
This commit is contained in:
Savannah Ostrowski 2023-10-29 13:53:25 -07:00 • committed by GitHub
parent 4d6bdf8aab
commit 4a929d432b
No known key found for this signature in database
GPG key ID: 4AEE18F83AFDEB23
4 changed files with 26 additions and 19 deletions
Download patch file Download diff file Expand all files Collapse all files

5
Include/internal/pycore_optimizer.h
View file

@ -13,6 +13,11 @@ extern "C" {
int _Py_uop_analyze_and_optimize(PyCodeObject *code, int _Py_uop_analyze_and_optimize(PyCodeObject *code,
_PyUOpInstruction *trace, int trace_len, int curr_stackentries); _PyUOpInstruction *trace, int trace_len, int curr_stackentries);
extern PyTypeObject _PyCounterExecutor_Type;
extern PyTypeObject _PyCounterOptimizer_Type;
extern PyTypeObject _PyDefaultOptimizer_Type;
extern PyTypeObject _PyUOpExecutor_Type;
extern PyTypeObject _PyUOpOptimizer_Type;
#ifdef __cplusplus #ifdef __cplusplus
} }

6
Objects/object.c
View file

@ -13,6 +13,7 @@
#include "pycore_memoryobject.h" // _PyManagedBuffer_Type #include "pycore_memoryobject.h" // _PyManagedBuffer_Type
#include "pycore_namespace.h" // _PyNamespace_Type #include "pycore_namespace.h" // _PyNamespace_Type
#include "pycore_object.h" // PyAPI_DATA() _Py_SwappedOp definition #include "pycore_object.h" // PyAPI_DATA() _Py_SwappedOp definition
#include "pycore_optimizer.h" // _PyUOpExecutor_Type, _PyUOpOptimizer_Type, ...
#include "pycore_pyerrors.h" // _PyErr_Occurred() #include "pycore_pyerrors.h" // _PyErr_Occurred()
#include "pycore_pymem.h" // _PyMem_IsPtrFreed() #include "pycore_pymem.h" // _PyMem_IsPtrFreed()
#include "pycore_pystate.h" // _PyThreadState_GET() #include "pycore_pystate.h" // _PyThreadState_GET()
@ -2157,6 +2158,9 @@ static PyTypeObject* static_types[] = {
&_PyBufferWrapper_Type, &_PyBufferWrapper_Type,
&_PyContextTokenMissing_Type, &_PyContextTokenMissing_Type,
&_PyCoroWrapper_Type, &_PyCoroWrapper_Type,
&_PyCounterExecutor_Type,
&_PyCounterOptimizer_Type,
&_PyDefaultOptimizer_Type,
&_Py_GenericAliasIterType, &_Py_GenericAliasIterType,
&_PyHamtItems_Type, &_PyHamtItems_Type,
&_PyHamtKeys_Type, &_PyHamtKeys_Type,
@ -2176,6 +2180,8 @@ static PyTypeObject* static_types[] = {
&_PyPositionsIterator, &_PyPositionsIterator,
&_PyUnicodeASCIIIter_Type, &_PyUnicodeASCIIIter_Type,
&_PyUnion_Type, &_PyUnion_Type,
&_PyUOpExecutor_Type,
&_PyUOpOptimizer_Type,
&_PyWeakref_CallableProxyType, &_PyWeakref_CallableProxyType,
&_PyWeakref_ProxyType, &_PyWeakref_ProxyType,
&_PyWeakref_RefType, &_PyWeakref_RefType,

24
Python/optimizer.c
View file

@ -111,7 +111,7 @@ error_optimize(
return -1; return -1;
} }
static PyTypeObject DefaultOptimizer_Type = { PyTypeObject _PyDefaultOptimizer_Type = {
PyVarObject_HEAD_INIT(&PyType_Type, 0) PyVarObject_HEAD_INIT(&PyType_Type, 0)
.tp_name = "noop_optimizer", .tp_name = "noop_optimizer",
.tp_basicsize = sizeof(_PyOptimizerObject), .tp_basicsize = sizeof(_PyOptimizerObject),
@ -120,7 +120,7 @@ static PyTypeObject DefaultOptimizer_Type = {
}; };
_PyOptimizerObject _PyOptimizer_Default = { _PyOptimizerObject _PyOptimizer_Default = {
PyObject_HEAD_INIT(&DefaultOptimizer_Type) PyObject_HEAD_INIT(&_PyDefaultOptimizer_Type)
.optimize = error_optimize, .optimize = error_optimize,
.resume_threshold = UINT16_MAX, .resume_threshold = UINT16_MAX,
.backedge_threshold = UINT16_MAX, .backedge_threshold = UINT16_MAX,
@ -236,7 +236,7 @@ static PyMethodDef executor_methods[] = {
{ NULL, NULL }, { NULL, NULL },
}; };
static PyTypeObject CounterExecutor_Type = { PyTypeObject _PyCounterExecutor_Type = {
PyVarObject_HEAD_INIT(&PyType_Type, 0) PyVarObject_HEAD_INIT(&PyType_Type, 0)
.tp_name = "counting_executor", .tp_name = "counting_executor",
.tp_basicsize = sizeof(_PyCounterExecutorObject), .tp_basicsize = sizeof(_PyCounterExecutorObject),
@ -265,7 +265,7 @@ counter_optimize(
int Py_UNUSED(curr_stackentries) int Py_UNUSED(curr_stackentries)
) )
{ {
_PyCounterExecutorObject *executor = (_PyCounterExecutorObject *)_PyObject_New(&CounterExecutor_Type); _PyCounterExecutorObject *executor = (_PyCounterExecutorObject *)_PyObject_New(&_PyCounterExecutor_Type);
if (executor == NULL) { if (executor == NULL) {
return -1; return -1;
} }
@ -291,7 +291,7 @@ static PyMethodDef counter_optimizer_methods[] = {
{ NULL, NULL }, { NULL, NULL },
}; };
static PyTypeObject CounterOptimizer_Type = { PyTypeObject _PyCounterOptimizer_Type = {
PyVarObject_HEAD_INIT(&PyType_Type, 0) PyVarObject_HEAD_INIT(&PyType_Type, 0)
.tp_name = "Counter optimizer", .tp_name = "Counter optimizer",
.tp_basicsize = sizeof(_PyCounterOptimizerObject), .tp_basicsize = sizeof(_PyCounterOptimizerObject),
@ -304,9 +304,7 @@ static PyTypeObject CounterOptimizer_Type = {
PyObject * PyObject *
PyUnstable_Optimizer_NewCounter(void) PyUnstable_Optimizer_NewCounter(void)
{ {
PyType_Ready(&CounterExecutor_Type); _PyCounterOptimizerObject *opt = (_PyCounterOptimizerObject *)_PyObject_New(&_PyCounterOptimizer_Type);
PyType_Ready(&CounterOptimizer_Type);
_PyCounterOptimizerObject *opt = (_PyCounterOptimizerObject *)_PyObject_New(&CounterOptimizer_Type);
if (opt == NULL) { if (opt == NULL) {
return NULL; return NULL;
} }
@ -375,7 +373,7 @@ PySequenceMethods uop_as_sequence = {
.sq_item = (ssizeargfunc)uop_item, .sq_item = (ssizeargfunc)uop_item,
}; };
static PyTypeObject UOpExecutor_Type = { PyTypeObject _PyUOpExecutor_Type = {
PyVarObject_HEAD_INIT(&PyType_Type, 0) PyVarObject_HEAD_INIT(&PyType_Type, 0)
.tp_name = "uop_executor", .tp_name = "uop_executor",
.tp_basicsize = sizeof(_PyUOpExecutorObject) - sizeof(_PyUOpInstruction), .tp_basicsize = sizeof(_PyUOpExecutorObject) - sizeof(_PyUOpInstruction),
@ -929,7 +927,7 @@ uop_optimize(
trace_length = _Py_uop_analyze_and_optimize(code, trace, trace_length, curr_stackentries); trace_length = _Py_uop_analyze_and_optimize(code, trace, trace_length, curr_stackentries);
} }
trace_length = remove_unneeded_uops(trace, trace_length); trace_length = remove_unneeded_uops(trace, trace_length);
_PyUOpExecutorObject *executor = PyObject_NewVar(_PyUOpExecutorObject, &UOpExecutor_Type, trace_length); _PyUOpExecutorObject *executor = PyObject_NewVar(_PyUOpExecutorObject, &_PyUOpExecutor_Type, trace_length);
if (executor == NULL) { if (executor == NULL) {
return -1; return -1;
} }
@ -946,7 +944,7 @@ uop_opt_dealloc(PyObject *self) {
PyObject_Free(self); PyObject_Free(self);
} }
static PyTypeObject UOpOptimizer_Type = { PyTypeObject _PyUOpOptimizer_Type = {
PyVarObject_HEAD_INIT(&PyType_Type, 0) PyVarObject_HEAD_INIT(&PyType_Type, 0)
.tp_name = "uop_optimizer", .tp_name = "uop_optimizer",
.tp_basicsize = sizeof(_PyOptimizerObject), .tp_basicsize = sizeof(_PyOptimizerObject),
@ -958,9 +956,7 @@ static PyTypeObject UOpOptimizer_Type = {
PyObject * PyObject *
PyUnstable_Optimizer_NewUOpOptimizer(void) PyUnstable_Optimizer_NewUOpOptimizer(void)
{ {
PyType_Ready(&UOpExecutor_Type); _PyOptimizerObject *opt = PyObject_New(_PyOptimizerObject, &_PyUOpOptimizer_Type);
PyType_Ready(&UOpOptimizer_Type);
_PyOptimizerObject *opt = PyObject_New(_PyOptimizerObject, &UOpOptimizer_Type);
if (opt == NULL) { if (opt == NULL) {
return NULL; return NULL;
} }

10
Tools/c-analyzer/cpython/ignored.tsv
View file

@ -374,11 +374,11 @@ Python/sysmodule.c - perf_map_state -
Python/sysmodule.c - _PySys_ImplCacheTag - Python/sysmodule.c - _PySys_ImplCacheTag -
Python/sysmodule.c - _PySys_ImplName - Python/sysmodule.c - _PySys_ImplName -
Python/sysmodule.c - whatstrings - Python/sysmodule.c - whatstrings -
Python/optimizer.c - DefaultOptimizer_Type - Python/optimizer.c - _PyDefaultOptimizer_Type -
Python/optimizer.c - CounterExecutor_Type - Python/optimizer.c - _PyCounterExecutor_Type -
Python/optimizer.c - CounterOptimizer_Type - Python/optimizer.c - _PyCounterOptimizer_Type -
Python/optimizer.c - UOpExecutor_Type - Python/optimizer.c - _PyUOpExecutor_Type -
Python/optimizer.c - UOpOptimizer_Type - Python/optimizer.c - _PyUOpOptimizer_Type -
Python/optimizer.c - _PyOptimizer_Default - Python/optimizer.c - _PyOptimizer_Default -
##----------------------- ##-----------------------

Can't render this file because it has a wrong number of fields in line 4.