diff options
| author | Guido van Rossum <guido@python.org> | 2023-08-28 17:17:00 (GMT) |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2023-08-28 17:17:00 (GMT) |
| commit | 47d7eba889bc03884744f978f5f8612380363332 (patch) | |
| tree | 96f61d3955357b6b3ff5152e275f3accaadad293 /Python | |
| parent | 4116592b6f014a2720e9b09e2c8dec4bf4b4cd8f (diff) | |
| download | cpython-47d7eba889bc03884744f978f5f8612380363332.zip cpython-47d7eba889bc03884744f978f5f8612380363332.tar.gz cpython-47d7eba889bc03884744f978f5f8612380363332.tar.bz2 | |
gh-108487: Move assert(self != NULL) down beyond DEOPT_IF() (#108510)
Diffstat (limited to 'Python')
| -rw-r--r-- | Python/bytecodes.c | 2 | ||||
| -rw-r--r-- | Python/generated_cases.c.h | 2 |
2 files changed, 2 insertions, 2 deletions
diff --git a/Python/bytecodes.c b/Python/bytecodes.c index a55460a..93926c0 100644 --- a/Python/bytecodes.c +++ b/Python/bytecodes.c @@ -3346,9 +3346,9 @@ dummy_func( inst(CALL_NO_KW_LIST_APPEND, (unused/1, unused/2, callable, self, args[oparg] -- unused)) { ASSERT_KWNAMES_IS_NULL(); assert(oparg == 1); - assert(self != NULL); PyInterpreterState *interp = tstate->interp; DEOPT_IF(callable != interp->callable_cache.list_append, CALL); + assert(self != NULL); DEOPT_IF(!PyList_Check(self), CALL); STAT_INC(CALL, hit); if (_PyList_AppendTakeRef((PyListObject *)self, args[0]) < 0) { diff --git a/Python/generated_cases.c.h b/Python/generated_cases.c.h index 3f46f1a..5940c18 100644 --- a/Python/generated_cases.c.h +++ b/Python/generated_cases.c.h @@ -4412,9 +4412,9 @@ callable = stack_pointer[-2 - oparg]; ASSERT_KWNAMES_IS_NULL(); assert(oparg == 1); - assert(self != NULL); PyInterpreterState *interp = tstate->interp; DEOPT_IF(callable != interp->callable_cache.list_append, CALL); + assert(self != NULL); DEOPT_IF(!PyList_Check(self), CALL); STAT_INC(CALL, hit); if (_PyList_AppendTakeRef((PyListObject *)self, args[0]) < 0) { |