Guido van Rossum | d8faa36 | 2007-04-27 19:54:29 +0000 | [diff] [blame] | 1 | /* Peephole optimizations for bytecode compiler. */ |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 2 | |
| 3 | #include "Python.h" |
| 4 | |
| 5 | #include "Python-ast.h" |
| 6 | #include "node.h" |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 7 | #include "ast.h" |
| 8 | #include "code.h" |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 9 | #include "symtable.h" |
| 10 | #include "opcode.h" |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 11 | #include "wordcode_helpers.h" |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 12 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 13 | #define UNCONDITIONAL_JUMP(op) (op==JUMP_ABSOLUTE || op==JUMP_FORWARD) |
Jeffrey Yasskin | 9de7ec7 | 2009-02-25 02:25:04 +0000 | [diff] [blame] | 14 | #define CONDITIONAL_JUMP(op) (op==POP_JUMP_IF_FALSE || op==POP_JUMP_IF_TRUE \ |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 15 | || op==JUMP_IF_FALSE_OR_POP || op==JUMP_IF_TRUE_OR_POP) |
Serhiy Storchaka | 520b7ae | 2018-02-22 23:33:30 +0200 | [diff] [blame] | 16 | #define ABSOLUTE_JUMP(op) (op==JUMP_ABSOLUTE \ |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 17 | || op==POP_JUMP_IF_FALSE || op==POP_JUMP_IF_TRUE \ |
| 18 | || op==JUMP_IF_FALSE_OR_POP || op==JUMP_IF_TRUE_OR_POP) |
Jeffrey Yasskin | 9de7ec7 | 2009-02-25 02:25:04 +0000 | [diff] [blame] | 19 | #define JUMPS_ON_TRUE(op) (op==POP_JUMP_IF_TRUE || op==JUMP_IF_TRUE_OR_POP) |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 20 | #define GETJUMPTGT(arr, i) (get_arg(arr, i) / sizeof(_Py_CODEUNIT) + \ |
| 21 | (ABSOLUTE_JUMP(_Py_OPCODE(arr[i])) ? 0 : i+1)) |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 22 | #define ISBASICBLOCK(blocks, start, end) \ |
| 23 | (blocks[start]==blocks[end]) |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 24 | |
Antoine Pitrou | 17b880a | 2011-03-11 17:27:02 +0100 | [diff] [blame] | 25 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 26 | /* Scans back N consecutive LOAD_CONST instructions, skipping NOPs, |
| 27 | returns index of the Nth last's LOAD_CONST's EXTENDED_ARG prefix. |
| 28 | Callers are responsible to check CONST_STACK_LEN beforehand. |
| 29 | */ |
| 30 | static Py_ssize_t |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 31 | lastn_const_start(const _Py_CODEUNIT *codestr, Py_ssize_t i, Py_ssize_t n) |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 32 | { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 33 | assert(n > 0); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 34 | for (;;) { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 35 | i--; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 36 | assert(i >= 0); |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 37 | if (_Py_OPCODE(codestr[i]) == LOAD_CONST) { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 38 | if (!--n) { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 39 | while (i > 0 && _Py_OPCODE(codestr[i-1]) == EXTENDED_ARG) { |
| 40 | i--; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 41 | } |
| 42 | return i; |
| 43 | } |
| 44 | } |
| 45 | else { |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 46 | assert(_Py_OPCODE(codestr[i]) == EXTENDED_ARG); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 47 | } |
| 48 | } |
| 49 | } |
Antoine Pitrou | 17b880a | 2011-03-11 17:27:02 +0100 | [diff] [blame] | 50 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 51 | /* Scans through EXTENDED ARGs, seeking the index of the effective opcode */ |
| 52 | static Py_ssize_t |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 53 | find_op(const _Py_CODEUNIT *codestr, Py_ssize_t i) |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 54 | { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 55 | while (_Py_OPCODE(codestr[i]) == EXTENDED_ARG) { |
| 56 | i++; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 57 | } |
| 58 | return i; |
| 59 | } |
Antoine Pitrou | 17b880a | 2011-03-11 17:27:02 +0100 | [diff] [blame] | 60 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 61 | /* Given the index of the effective opcode, |
| 62 | scan back to construct the oparg with EXTENDED_ARG */ |
| 63 | static unsigned int |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 64 | get_arg(const _Py_CODEUNIT *codestr, Py_ssize_t i) |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 65 | { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 66 | _Py_CODEUNIT word; |
| 67 | unsigned int oparg = _Py_OPARG(codestr[i]); |
| 68 | if (i >= 1 && _Py_OPCODE(word = codestr[i-1]) == EXTENDED_ARG) { |
| 69 | oparg |= _Py_OPARG(word) << 8; |
| 70 | if (i >= 2 && _Py_OPCODE(word = codestr[i-2]) == EXTENDED_ARG) { |
| 71 | oparg |= _Py_OPARG(word) << 16; |
| 72 | if (i >= 3 && _Py_OPCODE(word = codestr[i-3]) == EXTENDED_ARG) { |
| 73 | oparg |= _Py_OPARG(word) << 24; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 74 | } |
| 75 | } |
| 76 | } |
| 77 | return oparg; |
| 78 | } |
| 79 | |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 80 | /* Fill the region with NOPs. */ |
| 81 | static void |
| 82 | fill_nops(_Py_CODEUNIT *codestr, Py_ssize_t start, Py_ssize_t end) |
| 83 | { |
| 84 | memset(codestr + start, NOP, (end - start) * sizeof(_Py_CODEUNIT)); |
| 85 | } |
| 86 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 87 | /* Given the index of the effective opcode, |
| 88 | attempt to replace the argument, taking into account EXTENDED_ARG. |
| 89 | Returns -1 on failure, or the new op index on success */ |
| 90 | static Py_ssize_t |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 91 | set_arg(_Py_CODEUNIT *codestr, Py_ssize_t i, unsigned int oparg) |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 92 | { |
| 93 | unsigned int curarg = get_arg(codestr, i); |
| 94 | int curilen, newilen; |
| 95 | if (curarg == oparg) |
| 96 | return i; |
| 97 | curilen = instrsize(curarg); |
| 98 | newilen = instrsize(oparg); |
| 99 | if (curilen < newilen) { |
| 100 | return -1; |
| 101 | } |
| 102 | |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 103 | write_op_arg(codestr + i + 1 - curilen, _Py_OPCODE(codestr[i]), oparg, newilen); |
| 104 | fill_nops(codestr, i + 1 - curilen + newilen, i + 1); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 105 | return i-curilen+newilen; |
| 106 | } |
| 107 | |
| 108 | /* Attempt to write op/arg at end of specified region of memory. |
| 109 | Preceding memory in the region is overwritten with NOPs. |
| 110 | Returns -1 on failure, op index on success */ |
| 111 | static Py_ssize_t |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 112 | copy_op_arg(_Py_CODEUNIT *codestr, Py_ssize_t i, unsigned char op, |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 113 | unsigned int oparg, Py_ssize_t maxi) |
| 114 | { |
| 115 | int ilen = instrsize(oparg); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 116 | if (i + ilen > maxi) { |
| 117 | return -1; |
| 118 | } |
| 119 | write_op_arg(codestr + maxi - ilen, op, oparg, ilen); |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 120 | fill_nops(codestr, i, maxi - ilen); |
| 121 | return maxi - 1; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 122 | } |
| 123 | |
| 124 | /* Replace LOAD_CONST c1, LOAD_CONST c2 ... LOAD_CONST cn, BUILD_TUPLE n |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 125 | with LOAD_CONST (c1, c2, ... cn). |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 126 | The consts table must still be in list form so that the |
| 127 | new constant (c1, c2, ... cn) can be appended. |
| 128 | Called with codestr pointing to the first LOAD_CONST. |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 129 | */ |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 130 | static Py_ssize_t |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 131 | fold_tuple_on_constants(_Py_CODEUNIT *codestr, Py_ssize_t c_start, |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 132 | Py_ssize_t opcode_end, PyObject *consts, int n) |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 133 | { |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 134 | /* Pre-conditions */ |
| 135 | assert(PyList_CheckExact(consts)); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 136 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 137 | /* Buildup new tuple of constants */ |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 138 | PyObject *newconst = PyTuple_New(n); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 139 | if (newconst == NULL) { |
| 140 | return -1; |
| 141 | } |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 142 | |
| 143 | for (Py_ssize_t i = 0, pos = c_start; i < n; i++, pos++) { |
| 144 | assert(pos < opcode_end); |
| 145 | pos = find_op(codestr, pos); |
| 146 | assert(_Py_OPCODE(codestr[pos]) == LOAD_CONST); |
| 147 | |
| 148 | unsigned int arg = get_arg(codestr, pos); |
| 149 | PyObject *constant = PyList_GET_ITEM(consts, arg); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 150 | Py_INCREF(constant); |
| 151 | PyTuple_SET_ITEM(newconst, i, constant); |
| 152 | } |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 153 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 154 | /* Append folded constant onto consts */ |
| 155 | if (PyList_Append(consts, newconst)) { |
| 156 | Py_DECREF(newconst); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 157 | return -1; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 158 | } |
| 159 | Py_DECREF(newconst); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 160 | |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 161 | return copy_op_arg(codestr, c_start, LOAD_CONST, |
| 162 | PyList_GET_SIZE(consts)-1, opcode_end); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 163 | } |
| 164 | |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 165 | static unsigned int * |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 166 | markblocks(_Py_CODEUNIT *code, Py_ssize_t len) |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 167 | { |
Serhiy Storchaka | 1a1ff29 | 2015-02-16 13:28:22 +0200 | [diff] [blame] | 168 | unsigned int *blocks = PyMem_New(unsigned int, len); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 169 | int i, j, opcode, blockcnt = 0; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 170 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 171 | if (blocks == NULL) { |
| 172 | PyErr_NoMemory(); |
| 173 | return NULL; |
| 174 | } |
| 175 | memset(blocks, 0, len*sizeof(int)); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 176 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 177 | /* Mark labels in the first pass */ |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 178 | for (i = 0; i < len; i++) { |
| 179 | opcode = _Py_OPCODE(code[i]); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 180 | switch (opcode) { |
| 181 | case FOR_ITER: |
| 182 | case JUMP_FORWARD: |
| 183 | case JUMP_IF_FALSE_OR_POP: |
| 184 | case JUMP_IF_TRUE_OR_POP: |
| 185 | case POP_JUMP_IF_FALSE: |
| 186 | case POP_JUMP_IF_TRUE: |
| 187 | case JUMP_ABSOLUTE: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 188 | case SETUP_FINALLY: |
| 189 | case SETUP_WITH: |
Yury Selivanov | 7544508 | 2015-05-11 22:57:16 -0400 | [diff] [blame] | 190 | case SETUP_ASYNC_WITH: |
Serhiy Storchaka | 520b7ae | 2018-02-22 23:33:30 +0200 | [diff] [blame] | 191 | case CALL_FINALLY: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 192 | j = GETJUMPTGT(code, i); |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 193 | assert(j < len); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 194 | blocks[j] = 1; |
| 195 | break; |
| 196 | } |
| 197 | } |
| 198 | /* Build block numbers in the second pass */ |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 199 | for (i = 0; i < len; i++) { |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 200 | blockcnt += blocks[i]; /* increment blockcnt over labels */ |
| 201 | blocks[i] = blockcnt; |
| 202 | } |
| 203 | return blocks; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 204 | } |
| 205 | |
| 206 | /* Perform basic peephole optimizations to components of a code object. |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 207 | The consts object should still be in list form to allow new constants |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 208 | to be appended. |
| 209 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 210 | To keep the optimizer simple, it bails when the lineno table has complex |
| 211 | encoding for gaps >= 255. |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 212 | |
Martin Panter | 46f5072 | 2016-05-26 05:35:26 +0000 | [diff] [blame] | 213 | Optimizations are restricted to simple transformations occurring within a |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 214 | single basic block. All transformations keep the code size the same or |
| 215 | smaller. For those that reduce size, the gaps are initially filled with |
| 216 | NOPs. Later those NOPs are removed and the jump addresses retargeted in |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 217 | a single pass. */ |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 218 | |
| 219 | PyObject * |
| 220 | PyCode_Optimize(PyObject *code, PyObject* consts, PyObject *names, |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 221 | PyObject *lnotab_obj) |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 222 | { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 223 | Py_ssize_t h, i, nexti, op_start, codelen, tgt; |
| 224 | unsigned int j, nops; |
| 225 | unsigned char opcode, nextop; |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 226 | _Py_CODEUNIT *codestr = NULL; |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 227 | unsigned char *lnotab; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 228 | unsigned int cum_orig_offset, last_offset; |
Serhiy Storchaka | 67559bf | 2015-02-16 21:13:24 +0200 | [diff] [blame] | 229 | Py_ssize_t tabsiz; |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 230 | // Count runs of consecutive LOAD_CONSTs |
| 231 | unsigned int cumlc = 0, lastlc = 0; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 232 | unsigned int *blocks = NULL; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 233 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 234 | /* Bail out if an exception is set */ |
| 235 | if (PyErr_Occurred()) |
| 236 | goto exitError; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 237 | |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 238 | /* Bypass optimization when the lnotab table is too complex */ |
| 239 | assert(PyBytes_Check(lnotab_obj)); |
| 240 | lnotab = (unsigned char*)PyBytes_AS_STRING(lnotab_obj); |
| 241 | tabsiz = PyBytes_GET_SIZE(lnotab_obj); |
| 242 | assert(tabsiz == 0 || Py_REFCNT(lnotab_obj) == 1); |
| 243 | if (memchr(lnotab, 255, tabsiz) != NULL) { |
| 244 | /* 255 value are used for multibyte bytecode instructions */ |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 245 | goto exitUnchanged; |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 246 | } |
| 247 | /* Note: -128 and 127 special values for line number delta are ok, |
| 248 | the peephole optimizer doesn't modify line numbers. */ |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 249 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 250 | assert(PyBytes_Check(code)); |
| 251 | codelen = PyBytes_GET_SIZE(code); |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 252 | assert(codelen % sizeof(_Py_CODEUNIT) == 0); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 253 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 254 | /* Make a modifiable copy of the code string */ |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 255 | codestr = (_Py_CODEUNIT *)PyMem_Malloc(codelen); |
Victor Stinner | e0af3a8 | 2013-07-09 00:32:04 +0200 | [diff] [blame] | 256 | if (codestr == NULL) { |
| 257 | PyErr_NoMemory(); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 258 | goto exitError; |
Victor Stinner | e0af3a8 | 2013-07-09 00:32:04 +0200 | [diff] [blame] | 259 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 260 | memcpy(codestr, PyBytes_AS_STRING(code), codelen); |
| 261 | codelen /= sizeof(_Py_CODEUNIT); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 262 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 263 | blocks = markblocks(codestr, codelen); |
| 264 | if (blocks == NULL) |
| 265 | goto exitError; |
| 266 | assert(PyList_Check(consts)); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 267 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 268 | for (i=find_op(codestr, 0) ; i<codelen ; i=nexti) { |
Serhiy Storchaka | a1e9ab3 | 2016-09-11 15:19:12 +0300 | [diff] [blame] | 269 | opcode = _Py_OPCODE(codestr[i]); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 270 | op_start = i; |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 271 | while (op_start >= 1 && _Py_OPCODE(codestr[op_start-1]) == EXTENDED_ARG) { |
| 272 | op_start--; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 273 | } |
| 274 | |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 275 | nexti = i + 1; |
| 276 | while (nexti < codelen && _Py_OPCODE(codestr[nexti]) == EXTENDED_ARG) |
| 277 | nexti++; |
| 278 | nextop = nexti < codelen ? _Py_OPCODE(codestr[nexti]) : 0; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 279 | |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 280 | lastlc = cumlc; |
| 281 | cumlc = 0; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 282 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 283 | switch (opcode) { |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 284 | /* Skip over LOAD_CONST trueconst |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 285 | POP_JUMP_IF_FALSE xx. This improves |
| 286 | "while 1" performance. */ |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 287 | case LOAD_CONST: |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 288 | cumlc = lastlc + 1; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 289 | if (nextop != POP_JUMP_IF_FALSE || |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 290 | !ISBASICBLOCK(blocks, op_start, i + 1) || |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 291 | !PyObject_IsTrue(PyList_GET_ITEM(consts, get_arg(codestr, i)))) |
| 292 | break; |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 293 | fill_nops(codestr, op_start, nexti + 1); |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 294 | cumlc = 0; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 295 | break; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 296 | |
Serhiy Storchaka | 15a8728 | 2017-12-14 20:24:31 +0200 | [diff] [blame] | 297 | /* Try to fold tuples of constants. |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 298 | Skip over BUILD_SEQN 1 UNPACK_SEQN 1. |
| 299 | Replace BUILD_SEQN 2 UNPACK_SEQN 2 with ROT2. |
| 300 | Replace BUILD_SEQN 3 UNPACK_SEQN 3 with ROT3 ROT2. */ |
| 301 | case BUILD_TUPLE: |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 302 | j = get_arg(codestr, i); |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 303 | if (j > 0 && lastlc >= j) { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 304 | h = lastn_const_start(codestr, op_start, j); |
Serhiy Storchaka | 15a8728 | 2017-12-14 20:24:31 +0200 | [diff] [blame] | 305 | if (ISBASICBLOCK(blocks, h, op_start)) { |
INADA Naoki | 87010e8 | 2017-12-18 15:52:54 +0900 | [diff] [blame] | 306 | h = fold_tuple_on_constants(codestr, h, i+1, consts, j); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 307 | break; |
| 308 | } |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 309 | } |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 310 | if (nextop != UNPACK_SEQUENCE || |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 311 | !ISBASICBLOCK(blocks, op_start, i + 1) || |
Serhiy Storchaka | 15a8728 | 2017-12-14 20:24:31 +0200 | [diff] [blame] | 312 | j != get_arg(codestr, nexti)) |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 313 | break; |
| 314 | if (j < 2) { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 315 | fill_nops(codestr, op_start, nexti + 1); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 316 | } else if (j == 2) { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 317 | codestr[op_start] = PACKOPARG(ROT_TWO, 0); |
| 318 | fill_nops(codestr, op_start + 1, nexti + 1); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 319 | } else if (j == 3) { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 320 | codestr[op_start] = PACKOPARG(ROT_THREE, 0); |
| 321 | codestr[op_start + 1] = PACKOPARG(ROT_TWO, 0); |
| 322 | fill_nops(codestr, op_start + 2, nexti + 1); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 323 | } |
| 324 | break; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 325 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 326 | /* Simplify conditional jump to conditional jump where the |
| 327 | result of the first test implies the success of a similar |
| 328 | test or the failure of the opposite test. |
| 329 | Arises in code like: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 330 | "a and b or c" |
| 331 | "(a and b) and c" |
Serhiy Storchaka | 36ff451 | 2017-06-11 14:50:22 +0300 | [diff] [blame] | 332 | "(a or b) or c" |
| 333 | "(a or b) and c" |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 334 | x:JUMP_IF_FALSE_OR_POP y y:JUMP_IF_FALSE_OR_POP z |
| 335 | --> x:JUMP_IF_FALSE_OR_POP z |
| 336 | x:JUMP_IF_FALSE_OR_POP y y:JUMP_IF_TRUE_OR_POP z |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 337 | --> x:POP_JUMP_IF_FALSE y+1 |
| 338 | where y+1 is the instruction following the second test. |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 339 | */ |
| 340 | case JUMP_IF_FALSE_OR_POP: |
| 341 | case JUMP_IF_TRUE_OR_POP: |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 342 | h = get_arg(codestr, i) / sizeof(_Py_CODEUNIT); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 343 | tgt = find_op(codestr, h); |
| 344 | |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 345 | j = _Py_OPCODE(codestr[tgt]); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 346 | if (CONDITIONAL_JUMP(j)) { |
Raymond Hettinger | 08eef3f | 2016-08-07 20:20:33 -0700 | [diff] [blame] | 347 | /* NOTE: all possible jumps here are absolute. */ |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 348 | if (JUMPS_ON_TRUE(j) == JUMPS_ON_TRUE(opcode)) { |
Raymond Hettinger | 08eef3f | 2016-08-07 20:20:33 -0700 | [diff] [blame] | 349 | /* The second jump will be taken iff the first is. |
| 350 | The current opcode inherits its target's |
| 351 | stack effect */ |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 352 | h = set_arg(codestr, i, get_arg(codestr, tgt)); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 353 | } else { |
Raymond Hettinger | 08eef3f | 2016-08-07 20:20:33 -0700 | [diff] [blame] | 354 | /* The second jump is not taken if the first is (so |
| 355 | jump past it), and all conditional jumps pop their |
| 356 | argument when they're not taken (so change the |
| 357 | first jump to pop its argument when it's taken). */ |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 358 | h = set_arg(codestr, i, (tgt + 1) * sizeof(_Py_CODEUNIT)); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 359 | j = opcode == JUMP_IF_TRUE_OR_POP ? |
| 360 | POP_JUMP_IF_TRUE : POP_JUMP_IF_FALSE; |
| 361 | } |
| 362 | |
| 363 | if (h >= 0) { |
| 364 | nexti = h; |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 365 | codestr[nexti] = PACKOPARG(j, _Py_OPARG(codestr[nexti])); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 366 | break; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 367 | } |
| 368 | } |
| 369 | /* Intentional fallthrough */ |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 370 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 371 | /* Replace jumps to unconditional jumps */ |
| 372 | case POP_JUMP_IF_FALSE: |
| 373 | case POP_JUMP_IF_TRUE: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 374 | case JUMP_FORWARD: |
| 375 | case JUMP_ABSOLUTE: |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 376 | h = GETJUMPTGT(codestr, i); |
| 377 | tgt = find_op(codestr, h); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 378 | /* Replace JUMP_* to a RETURN into just a RETURN */ |
| 379 | if (UNCONDITIONAL_JUMP(opcode) && |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 380 | _Py_OPCODE(codestr[tgt]) == RETURN_VALUE) { |
| 381 | codestr[op_start] = PACKOPARG(RETURN_VALUE, 0); |
| 382 | fill_nops(codestr, op_start + 1, i + 1); |
| 383 | } else if (UNCONDITIONAL_JUMP(_Py_OPCODE(codestr[tgt]))) { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 384 | j = GETJUMPTGT(codestr, tgt); |
| 385 | if (opcode == JUMP_FORWARD) { /* JMP_ABS can go backwards */ |
| 386 | opcode = JUMP_ABSOLUTE; |
| 387 | } else if (!ABSOLUTE_JUMP(opcode)) { |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 388 | if ((Py_ssize_t)j < i + 1) { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 389 | break; /* No backward relative jumps */ |
| 390 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 391 | j -= i + 1; /* Calc relative jump addr */ |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 392 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 393 | j *= sizeof(_Py_CODEUNIT); |
| 394 | copy_op_arg(codestr, op_start, opcode, j, i + 1); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 395 | } |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 396 | break; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 397 | |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 398 | /* Remove unreachable ops after RETURN */ |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 399 | case RETURN_VALUE: |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 400 | h = i + 1; |
Serhiy Storchaka | 520b7ae | 2018-02-22 23:33:30 +0200 | [diff] [blame] | 401 | /* END_FINALLY should be kept since it denotes the end of |
| 402 | the 'finally' block in frame_setlineno() in frameobject.c. |
| 403 | SETUP_FINALLY should be kept for balancing. |
| 404 | */ |
| 405 | while (h < codelen && ISBASICBLOCK(blocks, i, h) && |
| 406 | _Py_OPCODE(codestr[h]) != END_FINALLY) |
| 407 | { |
| 408 | if (_Py_OPCODE(codestr[h]) == SETUP_FINALLY) { |
| 409 | while (h > i + 1 && |
| 410 | _Py_OPCODE(codestr[h - 1]) == EXTENDED_ARG) |
| 411 | { |
| 412 | h--; |
| 413 | } |
| 414 | break; |
| 415 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 416 | h++; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 417 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 418 | if (h > i + 1) { |
Serhiy Storchaka | 7db3c48 | 2016-10-25 09:30:43 +0300 | [diff] [blame] | 419 | fill_nops(codestr, i + 1, h); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 420 | nexti = find_op(codestr, h); |
| 421 | } |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 422 | break; |
| 423 | } |
| 424 | } |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 425 | |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 426 | /* Fixup lnotab */ |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 427 | for (i = 0, nops = 0; i < codelen; i++) { |
Serhiy Storchaka | 67559bf | 2015-02-16 21:13:24 +0200 | [diff] [blame] | 428 | assert(i - nops <= INT_MAX); |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 429 | /* original code offset => new code offset */ |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 430 | blocks[i] = i - nops; |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 431 | if (_Py_OPCODE(codestr[i]) == NOP) |
| 432 | nops++; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 433 | } |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 434 | cum_orig_offset = 0; |
| 435 | last_offset = 0; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 436 | for (i=0 ; i < tabsiz ; i+=2) { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 437 | unsigned int offset_delta, new_offset; |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 438 | cum_orig_offset += lnotab[i]; |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 439 | assert(cum_orig_offset % sizeof(_Py_CODEUNIT) == 0); |
| 440 | new_offset = blocks[cum_orig_offset / sizeof(_Py_CODEUNIT)] * |
| 441 | sizeof(_Py_CODEUNIT); |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 442 | offset_delta = new_offset - last_offset; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 443 | assert(offset_delta <= 255); |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 444 | lnotab[i] = (unsigned char)offset_delta; |
| 445 | last_offset = new_offset; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 446 | } |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 447 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 448 | /* Remove NOPs and fixup jump targets */ |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 449 | for (op_start = i = h = 0; i < codelen; i++, op_start = i) { |
| 450 | j = _Py_OPARG(codestr[i]); |
| 451 | while (_Py_OPCODE(codestr[i]) == EXTENDED_ARG) { |
| 452 | i++; |
| 453 | j = j<<8 | _Py_OPARG(codestr[i]); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 454 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 455 | opcode = _Py_OPCODE(codestr[i]); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 456 | switch (opcode) { |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 457 | case NOP:continue; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 458 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 459 | case JUMP_ABSOLUTE: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 460 | case POP_JUMP_IF_FALSE: |
| 461 | case POP_JUMP_IF_TRUE: |
| 462 | case JUMP_IF_FALSE_OR_POP: |
| 463 | case JUMP_IF_TRUE_OR_POP: |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 464 | j = blocks[j / sizeof(_Py_CODEUNIT)] * sizeof(_Py_CODEUNIT); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 465 | break; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 466 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 467 | case FOR_ITER: |
| 468 | case JUMP_FORWARD: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 469 | case SETUP_FINALLY: |
| 470 | case SETUP_WITH: |
Yury Selivanov | 7544508 | 2015-05-11 22:57:16 -0400 | [diff] [blame] | 471 | case SETUP_ASYNC_WITH: |
Serhiy Storchaka | 520b7ae | 2018-02-22 23:33:30 +0200 | [diff] [blame] | 472 | case CALL_FINALLY: |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 473 | j = blocks[j / sizeof(_Py_CODEUNIT) + i + 1] - blocks[i] - 1; |
| 474 | j *= sizeof(_Py_CODEUNIT); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 475 | break; |
| 476 | } |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 477 | nexti = i - op_start + 1; |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 478 | if (instrsize(j) > nexti) |
| 479 | goto exitUnchanged; |
| 480 | /* If instrsize(j) < nexti, we'll emit EXTENDED_ARG 0 */ |
| 481 | write_op_arg(codestr + h, opcode, j, nexti); |
| 482 | h += nexti; |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 483 | } |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 484 | assert(h + (Py_ssize_t)nops == codelen); |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 485 | |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 486 | PyMem_Free(blocks); |
Serhiy Storchaka | ab87400 | 2016-09-11 13:48:15 +0300 | [diff] [blame] | 487 | code = PyBytes_FromStringAndSize((char *)codestr, h * sizeof(_Py_CODEUNIT)); |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 488 | PyMem_Free(codestr); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 489 | return code; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 490 | |
Alexandre Vassalotti | 6f82818 | 2009-07-21 02:51:58 +0000 | [diff] [blame] | 491 | exitError: |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 492 | code = NULL; |
Alexandre Vassalotti | 6f82818 | 2009-07-21 02:51:58 +0000 | [diff] [blame] | 493 | |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 494 | exitUnchanged: |
Serhiy Storchaka | b0f80b0 | 2016-05-24 09:15:14 +0300 | [diff] [blame] | 495 | Py_XINCREF(code); |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 496 | PyMem_Free(blocks); |
Victor Stinner | f3914eb | 2016-01-20 12:16:21 +0100 | [diff] [blame] | 497 | PyMem_Free(codestr); |
Antoine Pitrou | f95a1b3 | 2010-05-09 15:52:27 +0000 | [diff] [blame] | 498 | return code; |
Thomas Wouters | 00ee7ba | 2006-08-21 19:07:27 +0000 | [diff] [blame] | 499 | } |