]> granicus.if.org Git - python/commitdiff
bpo-37289: Remove 'if False' handling in the peephole optimizer (GH-14099)
authorPablo Galindo <Pablogsal@gmail.com>
Sat, 15 Jun 2019 14:58:00 +0000 (15:58 +0100)
committerGitHub <noreply@github.com>
Sat, 15 Jun 2019 14:58:00 +0000 (15:58 +0100)
Python/peephole.c

index d7b1dfc4d9c1419d2b1fbd8d59bdeded5d9b7f56..3e56e788b0079b5f70b4c46cd7e6a9b5f6123c47 100644 (file)
@@ -311,18 +311,12 @@ PyCode_Optimize(PyObject *code, PyObject* consts, PyObject *names,
                 }
                 PyObject* cnt = PyList_GET_ITEM(consts, get_arg(codestr, i));
                 int is_true = PyObject_IsTrue(cnt);
+                if (is_true == -1) {
+                    goto exitError;
+                }
                 if (is_true == 1) {
                     fill_nops(codestr, op_start, nexti + 1);
                     cumlc = 0;
-                } else if (is_true == 0) {
-                    if (i > 1 &&
-                        (_Py_OPCODE(codestr[i - 1]) == POP_JUMP_IF_TRUE ||
-                         _Py_OPCODE(codestr[i - 1]) == POP_JUMP_IF_FALSE)) {
-                        break;
-                    }
-                    h = get_arg(codestr, nexti) / sizeof(_Py_CODEUNIT);
-                    tgt = find_op(codestr, codelen, h);
-                    fill_nops(codestr, op_start, tgt);
                 }
                 break;