[Python-checkins] gh-106608: make uop trace variable length (#107531)
gvanrossum
webhook-mailer at python.org
Sat Aug 5 00:10:50 EDT 2023
https://github.com/python/cpython/commit/4e6fac7fcc31fc6198fcddc612688b0a05ff7ae4
commit: 4e6fac7fcc31fc6198fcddc612688b0a05ff7ae4
branch: main
author: Ivin Lee <62840497+rdrf2838 at users.noreply.github.com>
committer: gvanrossum <gvanrossum at gmail.com>
date: 2023-08-04T21:10:46-07:00
summary:
gh-106608: make uop trace variable length (#107531)
Executors are now more like tuples.
files:
A Misc/NEWS.d/next/Core and Builtins/2023-08-01-09-41-36.gh-issue-106608.OFZogw.rst
M Include/cpython/optimizer.h
M Include/internal/pycore_uops.h
M Python/optimizer.c
diff --git a/Include/cpython/optimizer.h b/Include/cpython/optimizer.h
index 2260501bfd608..da34ec1882a53 100644
--- a/Include/cpython/optimizer.h
+++ b/Include/cpython/optimizer.h
@@ -12,7 +12,7 @@ typedef struct {
} _PyVMData;
typedef struct _PyExecutorObject {
- PyObject_HEAD
+ PyObject_VAR_HEAD
/* WARNING: execute consumes a reference to self. This is necessary to allow executors to tail call into each other. */
struct _PyInterpreterFrame *(*execute)(struct _PyExecutorObject *self, struct _PyInterpreterFrame *frame, PyObject **stack_pointer);
_PyVMData vm_data; /* Used by the VM, but opaque to the optimizer */
diff --git a/Include/internal/pycore_uops.h b/Include/internal/pycore_uops.h
index edb141cc79f75..57a5970353b36 100644
--- a/Include/internal/pycore_uops.h
+++ b/Include/internal/pycore_uops.h
@@ -18,7 +18,7 @@ typedef struct {
typedef struct {
_PyExecutorObject base;
- _PyUOpInstruction trace[_Py_UOP_MAX_TRACE_LENGTH]; // TODO: variable length
+ _PyUOpInstruction trace[1];
} _PyUOpExecutorObject;
_PyInterpreterFrame *_PyUopExecute(
diff --git a/Misc/NEWS.d/next/Core and Builtins/2023-08-01-09-41-36.gh-issue-106608.OFZogw.rst b/Misc/NEWS.d/next/Core and Builtins/2023-08-01-09-41-36.gh-issue-106608.OFZogw.rst
new file mode 100644
index 0000000000000..20d43a7c4f754
--- /dev/null
+++ b/Misc/NEWS.d/next/Core and Builtins/2023-08-01-09-41-36.gh-issue-106608.OFZogw.rst
@@ -0,0 +1 @@
+Make ``_PyUOpExecutorObject`` variable length.
diff --git a/Python/optimizer.c b/Python/optimizer.c
index 238ab02d09faa..d2ed8dfcd2cff 100644
--- a/Python/optimizer.c
+++ b/Python/optimizer.c
@@ -320,13 +320,7 @@ uop_name(int index) {
static Py_ssize_t
uop_len(_PyUOpExecutorObject *self)
{
- int count = 0;
- for (; count < _Py_UOP_MAX_TRACE_LENGTH; count++) {
- if (self->trace[count].opcode == 0) {
- break;
- }
- }
- return count;
+ return Py_SIZE(self);
}
static PyObject *
@@ -368,8 +362,8 @@ PySequenceMethods uop_as_sequence = {
static PyTypeObject UOpExecutor_Type = {
PyVarObject_HEAD_INIT(&PyType_Type, 0)
.tp_name = "uop_executor",
- .tp_basicsize = sizeof(_PyUOpExecutorObject),
- .tp_itemsize = 0,
+ .tp_basicsize = sizeof(_PyUOpExecutorObject) - sizeof(_PyUOpInstruction),
+ .tp_itemsize = sizeof(_PyUOpInstruction),
.tp_flags = Py_TPFLAGS_DEFAULT | Py_TPFLAGS_DISALLOW_INSTANTIATION,
.tp_dealloc = (destructor)uop_dealloc,
.tp_as_sequence = &uop_as_sequence,
@@ -699,15 +693,12 @@ uop_optimize(
return trace_length;
}
OBJECT_STAT_INC(optimization_traces_created);
- _PyUOpExecutorObject *executor = PyObject_New(_PyUOpExecutorObject, &UOpExecutor_Type);
+ _PyUOpExecutorObject *executor = PyObject_NewVar(_PyUOpExecutorObject, &UOpExecutor_Type, trace_length);
if (executor == NULL) {
return -1;
}
executor->base.execute = _PyUopExecute;
memcpy(executor->trace, trace, trace_length * sizeof(_PyUOpInstruction));
- if (trace_length < _Py_UOP_MAX_TRACE_LENGTH) {
- executor->trace[trace_length].opcode = 0; // Sentinel
- }
*exec_ptr = (_PyExecutorObject *)executor;
return 1;
}
More information about the Python-checkins
mailing list