diff options
author | Pablo Galindo <Pablogsal@gmail.com> | 2019-05-17 11:37:08 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2019-05-17 11:37:08 +0100 |
commit | af8646c8054d0f4180a2013383039b6a472f9698 (patch) | |
tree | ceafae5989deb1ab62b8237593bd8752ab600e39 /Python/peephole.c | |
parent | a8b46944d72bba6dc76260ed61da5c78d3f9d9c0 (diff) | |
download | cpython-git-af8646c8054d0f4180a2013383039b6a472f9698.tar.gz |
bpo-1875: Raise SyntaxError in invalid blocks that will be optimised away (GH-13332)
Move the check for dead conditionals (if 0) to the peephole optimizer
and make sure that the code block is still compiled to report any
existing syntax errors within.
Diffstat (limited to 'Python/peephole.c')
-rw-r--r-- | Python/peephole.c | 16 |
1 files changed, 12 insertions, 4 deletions
diff --git a/Python/peephole.c b/Python/peephole.c index cc244aa433..1ce3535626 100644 --- a/Python/peephole.c +++ b/Python/peephole.c @@ -302,11 +302,19 @@ PyCode_Optimize(PyObject *code, PyObject* consts, PyObject *names, case LOAD_CONST: cumlc = lastlc + 1; if (nextop != POP_JUMP_IF_FALSE || - !ISBASICBLOCK(blocks, op_start, i + 1) || - !PyObject_IsTrue(PyList_GET_ITEM(consts, get_arg(codestr, i)))) + !ISBASICBLOCK(blocks, op_start, i + 1)) { break; - fill_nops(codestr, op_start, nexti + 1); - cumlc = 0; + } + PyObject* cnt = PyList_GET_ITEM(consts, get_arg(codestr, i)); + int is_true = PyObject_IsTrue(cnt); + if (is_true == 1) { + fill_nops(codestr, op_start, nexti + 1); + cumlc = 0; + } else if (is_true == 0) { + h = get_arg(codestr, nexti) / sizeof(_Py_CODEUNIT); + tgt = find_op(codestr, codelen, h); + fill_nops(codestr, op_start, tgt); + } break; /* Try to fold tuples of constants. |