diff options
author | Guido van Rossum <guido@python.org> | 2023-11-20 10:08:53 -0800 |
---|---|---|
committer | GitHub <noreply@github.com> | 2023-11-20 10:08:53 -0800 |
commit | 1995955173737bcb009dbacaeff7821b4d744148 (patch) | |
tree | 96a6c3f5901c4ed2c4cf5fa205231f7216a6e9c1 /Python/executor_cases.c.h | |
parent | d59feb5dbe5395615d06c30a95e6a6a9b7681d4d (diff) | |
download | cpython-1995955173737bcb009dbacaeff7821b4d744148.tar.gz cpython-1995955173737bcb009dbacaeff7821b4d744148.zip |
gh-106529: Make FOR_ITER a viable uop (#112134)
This uses the new mechanism whereby certain uops
are replaced by others during translation,
using the `_PyUop_Replacements` table.
We further special-case the `_FOR_ITER_TIER_TWO` uop
to update the deoptimization target to point
just past the corresponding `END_FOR` opcode.
Two tiny code cleanups are also part of this PR.
Diffstat (limited to 'Python/executor_cases.c.h')
-rw-r--r-- | Python/executor_cases.c.h | 25 |
1 files changed, 25 insertions, 0 deletions
diff --git a/Python/executor_cases.c.h b/Python/executor_cases.c.h index 4e29fb9f0fa..ae662b20e44 100644 --- a/Python/executor_cases.c.h +++ b/Python/executor_cases.c.h @@ -2101,6 +2101,31 @@ break; } + case _FOR_ITER_TIER_TWO: { + PyObject *iter; + PyObject *next; + iter = stack_pointer[-1]; + /* before: [iter]; after: [iter, iter()] *or* [] (and jump over END_FOR.) */ + next = (*Py_TYPE(iter)->tp_iternext)(iter); + if (next == NULL) { + if (_PyErr_Occurred(tstate)) { + if (!_PyErr_ExceptionMatches(tstate, PyExc_StopIteration)) { + GOTO_ERROR(error); + } + _PyErr_Clear(tstate); + } + /* iterator ended normally */ + Py_DECREF(iter); + STACK_SHRINK(1); + /* The translator sets the deopt target just past END_FOR */ + DEOPT_IF(true, _FOR_ITER_TIER_TWO); + } + // Common case: no jump, leave it to the code generator + STACK_GROW(1); + stack_pointer[-1] = next; + break; + } + case _ITER_CHECK_LIST: { PyObject *iter; iter = stack_pointer[-1]; |