2014-05-03 18:27:38 -04:00
|
|
|
/*
|
|
|
|
* This file is part of the Micro Python project, http://micropython.org/
|
|
|
|
*
|
|
|
|
* The MIT License (MIT)
|
|
|
|
*
|
|
|
|
* Copyright (c) 2013, 2014 Damien P. George
|
2014-05-13 01:44:45 -04:00
|
|
|
* Copyright (c) 2014 Paul Sokolovsky
|
2014-05-03 18:27:38 -04:00
|
|
|
*
|
|
|
|
* Permission is hereby granted, free of charge, to any person obtaining a copy
|
|
|
|
* of this software and associated documentation files (the "Software"), to deal
|
|
|
|
* in the Software without restriction, including without limitation the rights
|
|
|
|
* to use, copy, modify, merge, publish, distribute, sublicense, and/or sell
|
|
|
|
* copies of the Software, and to permit persons to whom the Software is
|
|
|
|
* furnished to do so, subject to the following conditions:
|
|
|
|
*
|
|
|
|
* The above copyright notice and this permission notice shall be included in
|
|
|
|
* all copies or substantial portions of the Software.
|
|
|
|
*
|
|
|
|
* THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
|
|
|
|
* IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
|
|
|
|
* FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
|
|
|
|
* AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
|
|
|
|
* LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
|
|
|
|
* OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN
|
|
|
|
* THE SOFTWARE.
|
|
|
|
*/
|
|
|
|
|
2013-10-04 14:53:11 -04:00
|
|
|
#include <stdio.h>
|
2014-03-30 10:35:53 -04:00
|
|
|
#include <string.h>
|
2013-10-04 14:53:11 -04:00
|
|
|
#include <assert.h>
|
2014-05-30 20:41:08 -04:00
|
|
|
#include <alloca.h>
|
2013-10-04 14:53:11 -04:00
|
|
|
|
2014-05-02 10:47:01 -04:00
|
|
|
#include "mpconfig.h"
|
2013-10-15 17:25:17 -04:00
|
|
|
#include "nlr.h"
|
2013-10-04 14:53:11 -04:00
|
|
|
#include "misc.h"
|
2014-01-21 16:40:13 -05:00
|
|
|
#include "qstr.h"
|
2013-12-21 13:17:45 -05:00
|
|
|
#include "obj.h"
|
2014-04-13 06:04:33 -04:00
|
|
|
#include "emitglue.h"
|
2013-10-04 14:53:11 -04:00
|
|
|
#include "runtime.h"
|
2013-12-21 13:17:45 -05:00
|
|
|
#include "bc0.h"
|
2013-10-10 17:06:54 -04:00
|
|
|
#include "bc.h"
|
2014-03-26 11:36:12 -04:00
|
|
|
#include "objgenerator.h"
|
2013-10-04 14:53:11 -04:00
|
|
|
|
2014-05-31 09:50:46 -04:00
|
|
|
// With these macros you can tune the maximum number of function state bytes
|
2014-04-24 08:52:06 -04:00
|
|
|
// that will be allocated on the stack. Any function that needs more
|
|
|
|
// than this will use the heap.
|
2014-06-01 07:06:17 -04:00
|
|
|
#define VM_MAX_STATE_ON_STACK (10 * sizeof(machine_uint_t))
|
2014-04-24 08:52:06 -04:00
|
|
|
|
2014-04-10 12:21:34 -04:00
|
|
|
#define DETECT_VM_STACK_OVERFLOW (0)
|
2014-04-22 20:40:24 -04:00
|
|
|
#if 0
|
2014-05-10 05:36:38 -04:00
|
|
|
#define TRACE(ip) mp_bytecode_print2(ip, 1);
|
2014-04-22 20:40:24 -04:00
|
|
|
#else
|
|
|
|
#define TRACE(ip)
|
|
|
|
#endif
|
2014-04-10 12:21:34 -04:00
|
|
|
|
2014-01-31 12:45:15 -05:00
|
|
|
// Value stack grows up (this makes it incompatible with native C stack, but
|
|
|
|
// makes sure that arguments to functions are in natural order arg1..argN
|
|
|
|
// (Python semantics mandates left-to-right evaluation order, including for
|
|
|
|
// function arguments). Stack pointer is pre-incremented and points at the
|
|
|
|
// top element.
|
|
|
|
// Exception stack also grows up, top element is also pointed at.
|
|
|
|
|
2014-01-31 17:55:05 -05:00
|
|
|
// Exception stack unwind reasons (WHY_* in CPython-speak)
|
2014-02-01 15:08:18 -05:00
|
|
|
// TODO perhaps compress this to RETURN=0, JUMP>0, with number of unwinds
|
|
|
|
// left to do encoded in the JUMP number
|
2014-01-31 17:55:05 -05:00
|
|
|
typedef enum {
|
|
|
|
UNWIND_RETURN = 1,
|
2014-02-01 15:08:18 -05:00
|
|
|
UNWIND_JUMP,
|
2014-01-31 17:55:05 -05:00
|
|
|
} mp_unwind_reason_t;
|
|
|
|
|
2014-04-13 06:04:33 -04:00
|
|
|
#define DECODE_UINT do { \
|
2014-02-18 14:21:22 -05:00
|
|
|
unum = 0; \
|
|
|
|
do { \
|
|
|
|
unum = (unum << 7) + (*ip & 0x7f); \
|
|
|
|
} while ((*ip++ & 0x80) != 0); \
|
2014-04-13 06:04:33 -04:00
|
|
|
} while (0)
|
2013-11-05 17:06:08 -05:00
|
|
|
#define DECODE_ULABEL do { unum = (ip[0] | (ip[1] << 8)); ip += 2; } while (0)
|
|
|
|
#define DECODE_SLABEL do { unum = (ip[0] | (ip[1] << 8)) - 0x8000; ip += 2; } while (0)
|
2014-05-25 17:58:04 -04:00
|
|
|
#define DECODE_QSTR qstr qst = 0; \
|
2014-02-18 14:21:22 -05:00
|
|
|
do { \
|
|
|
|
qst = (qst << 7) + (*ip & 0x7f); \
|
2014-05-25 17:58:04 -04:00
|
|
|
} while ((*ip++ & 0x80) != 0)
|
2014-04-13 06:04:33 -04:00
|
|
|
#define DECODE_PTR do { \
|
|
|
|
ip = (byte*)(((machine_uint_t)ip + sizeof(machine_uint_t) - 1) & (~(sizeof(machine_uint_t) - 1))); /* align ip */ \
|
|
|
|
unum = *(machine_uint_t*)ip; \
|
|
|
|
ip += sizeof(machine_uint_t); \
|
|
|
|
} while (0)
|
2014-01-18 09:10:48 -05:00
|
|
|
#define PUSH(val) *++sp = (val)
|
|
|
|
#define POP() (*sp--)
|
2013-12-10 12:27:24 -05:00
|
|
|
#define TOP() (*sp)
|
|
|
|
#define SET_TOP(val) *sp = (val)
|
2013-10-04 14:53:11 -04:00
|
|
|
|
2014-03-29 17:16:27 -04:00
|
|
|
#define PUSH_EXC_BLOCK() \
|
2014-03-28 20:49:07 -04:00
|
|
|
DECODE_ULABEL; /* except labels are always forward */ \
|
|
|
|
++exc_sp; \
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
exc_sp->opcode = *code_state->ip; \
|
2014-03-28 20:49:07 -04:00
|
|
|
exc_sp->handler = ip + unum; \
|
|
|
|
exc_sp->val_sp = MP_TAGPTR_MAKE(sp, currently_in_except_block); \
|
2014-03-29 20:54:48 -04:00
|
|
|
exc_sp->prev_exc = MP_OBJ_NULL; \
|
2014-03-28 20:49:07 -04:00
|
|
|
currently_in_except_block = 0; /* in a try block now */
|
|
|
|
|
2014-03-29 17:16:27 -04:00
|
|
|
#define POP_EXC_BLOCK() \
|
|
|
|
currently_in_except_block = MP_TAGPTR_TAG(exc_sp->val_sp); /* restore previous state */ \
|
|
|
|
exc_sp--; /* pop back to previous exception handler */
|
|
|
|
|
2014-05-30 20:36:37 -04:00
|
|
|
mp_vm_return_kind_t mp_execute_bytecode(const byte *code, const mp_obj_t *args, uint n_args,
|
|
|
|
const mp_obj_t *args2, uint n_args2, mp_obj_t *ret) {
|
2014-03-27 06:55:21 -04:00
|
|
|
const byte *ip = code;
|
|
|
|
|
|
|
|
// get code info size, and skip line number table
|
|
|
|
machine_uint_t code_info_size = ip[0] | (ip[1] << 8) | (ip[2] << 16) | (ip[3] << 24);
|
|
|
|
ip += code_info_size;
|
|
|
|
|
2014-03-27 07:07:04 -04:00
|
|
|
// bytecode prelude: state size and exception stack size; 16 bit uints
|
|
|
|
machine_uint_t n_state = ip[0] | (ip[1] << 8);
|
|
|
|
machine_uint_t n_exc_stack = ip[2] | (ip[3] << 8);
|
|
|
|
ip += 4;
|
2014-03-27 06:55:21 -04:00
|
|
|
|
2014-01-18 09:10:48 -05:00
|
|
|
// allocate state for locals and stack
|
2014-04-10 12:21:34 -04:00
|
|
|
#if DETECT_VM_STACK_OVERFLOW
|
2014-04-10 12:59:44 -04:00
|
|
|
n_state += 1;
|
|
|
|
#endif
|
2014-01-18 09:10:48 -05:00
|
|
|
|
2014-05-31 09:50:46 -04:00
|
|
|
int state_size = n_state * sizeof(mp_obj_t) + n_exc_stack * sizeof(mp_exc_stack_t);
|
|
|
|
mp_code_state *code_state;
|
|
|
|
if (state_size > VM_MAX_STATE_ON_STACK) {
|
|
|
|
code_state = m_new_obj_var(mp_code_state, byte, state_size);
|
2014-05-30 20:41:08 -04:00
|
|
|
} else {
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state = alloca(sizeof(mp_code_state) + state_size);
|
2014-03-27 06:55:21 -04:00
|
|
|
}
|
2014-05-31 09:50:46 -04:00
|
|
|
|
|
|
|
code_state->code_info = code;
|
|
|
|
code_state->sp = &code_state->state[0] - 1;
|
|
|
|
code_state->exc_sp = (mp_exc_stack_t*)(code_state->state + n_state) - 1;
|
|
|
|
code_state->n_state = n_state;
|
2014-03-27 06:55:21 -04:00
|
|
|
|
2013-10-16 15:39:12 -04:00
|
|
|
// init args
|
2014-02-01 13:29:40 -05:00
|
|
|
for (uint i = 0; i < n_args; i++) {
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->state[n_state - 1 - i] = args[i];
|
2013-10-16 15:39:12 -04:00
|
|
|
}
|
2014-02-01 13:29:40 -05:00
|
|
|
for (uint i = 0; i < n_args2; i++) {
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->state[n_state - 1 - n_args - i] = args2[i];
|
2014-02-01 13:29:40 -05:00
|
|
|
}
|
2014-01-18 18:24:36 -05:00
|
|
|
|
2014-04-09 14:53:31 -04:00
|
|
|
// set rest of state to MP_OBJ_NULL
|
|
|
|
for (uint i = 0; i < n_state - n_args - n_args2; i++) {
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->state[i] = MP_OBJ_NULL;
|
2014-04-09 14:53:31 -04:00
|
|
|
}
|
|
|
|
|
2014-03-27 06:55:21 -04:00
|
|
|
// bytecode prelude: initialise closed over variables
|
|
|
|
for (uint n_local = *ip++; n_local > 0; n_local--) {
|
|
|
|
uint local_num = *ip++;
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->state[n_state - 1 - local_num] = mp_obj_new_cell(code_state->state[n_state - 1 - local_num]);
|
2013-12-30 17:32:17 -05:00
|
|
|
}
|
|
|
|
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->ip = ip;
|
|
|
|
|
2013-12-30 17:32:17 -05:00
|
|
|
// execute the byte code
|
2014-05-31 09:50:46 -04:00
|
|
|
mp_vm_return_kind_t vm_return_kind = mp_execute_bytecode2(code_state, MP_OBJ_NULL);
|
2014-02-15 17:55:00 -05:00
|
|
|
|
2014-04-10 12:21:34 -04:00
|
|
|
#if DETECT_VM_STACK_OVERFLOW
|
2014-05-31 11:14:54 -04:00
|
|
|
if (vm_return_kind == MP_VM_RETURN_NORMAL) {
|
2014-06-01 07:06:17 -04:00
|
|
|
if (code_state->sp < code_state->state) {
|
|
|
|
printf("VM stack underflow: " INT_FMT "\n", code_state->sp - code_state->state);
|
2014-05-31 11:14:54 -04:00
|
|
|
assert(0);
|
|
|
|
}
|
|
|
|
}
|
2014-04-10 12:59:44 -04:00
|
|
|
// We can't check the case when an exception is returned in state[n_state - 1]
|
|
|
|
// and there are no arguments, because in this case our detection slot may have
|
|
|
|
// been overwritten by the returned exception (which is allowed).
|
|
|
|
if (!(vm_return_kind == MP_VM_RETURN_EXCEPTION && n_args == 0 && n_args2 == 0)) {
|
|
|
|
// Just check to see that we have at least 1 null object left in the state.
|
|
|
|
bool overflow = true;
|
|
|
|
for (uint i = 0; i < n_state - n_args - n_args2; i++) {
|
2014-05-31 09:50:46 -04:00
|
|
|
if (code_state->state[i] == MP_OBJ_NULL) {
|
2014-04-10 12:59:44 -04:00
|
|
|
overflow = false;
|
|
|
|
break;
|
|
|
|
}
|
|
|
|
}
|
|
|
|
if (overflow) {
|
2014-06-01 07:06:17 -04:00
|
|
|
printf("VM stack overflow state=%p n_state+1=" UINT_FMT "\n", code_state->state, n_state);
|
2014-04-10 12:59:44 -04:00
|
|
|
assert(0);
|
|
|
|
}
|
2014-04-10 12:21:34 -04:00
|
|
|
}
|
|
|
|
#endif
|
|
|
|
|
2014-04-24 10:42:05 -04:00
|
|
|
mp_vm_return_kind_t ret_kind;
|
|
|
|
switch (vm_return_kind) {
|
|
|
|
case MP_VM_RETURN_NORMAL:
|
|
|
|
// return value is in *sp
|
2014-05-31 09:50:46 -04:00
|
|
|
*ret = *code_state->sp;
|
2014-04-24 10:42:05 -04:00
|
|
|
ret_kind = MP_VM_RETURN_NORMAL;
|
|
|
|
break;
|
|
|
|
|
|
|
|
case MP_VM_RETURN_EXCEPTION:
|
|
|
|
// return value is in state[n_state - 1]
|
2014-05-31 09:50:46 -04:00
|
|
|
*ret = code_state->state[n_state - 1];
|
2014-04-24 10:42:05 -04:00
|
|
|
ret_kind = MP_VM_RETURN_EXCEPTION;
|
|
|
|
break;
|
|
|
|
|
|
|
|
case MP_VM_RETURN_YIELD: // byte-code shouldn't yield
|
|
|
|
default:
|
|
|
|
assert(0);
|
|
|
|
*ret = mp_const_none;
|
|
|
|
ret_kind = MP_VM_RETURN_NORMAL;
|
2014-06-01 07:06:17 -04:00
|
|
|
break;
|
2014-04-24 10:42:05 -04:00
|
|
|
}
|
2014-04-24 08:52:06 -04:00
|
|
|
|
|
|
|
// free the state if it was allocated on the heap
|
2014-05-31 09:50:46 -04:00
|
|
|
if (state_size > VM_MAX_STATE_ON_STACK) {
|
|
|
|
m_del_var(mp_code_state, byte, state_size, code_state);
|
2014-04-24 08:52:06 -04:00
|
|
|
}
|
2014-04-24 10:42:05 -04:00
|
|
|
return ret_kind;
|
2013-10-16 15:39:12 -04:00
|
|
|
}
|
|
|
|
|
2014-01-18 09:10:48 -05:00
|
|
|
// fastn has items in reverse order (fastn[0] is local[0], fastn[-1] is local[1], etc)
|
|
|
|
// sp points to bottom of stack which grows up
|
2014-02-15 17:55:00 -05:00
|
|
|
// returns:
|
|
|
|
// MP_VM_RETURN_NORMAL, sp valid, return value in *sp
|
|
|
|
// MP_VM_RETURN_YIELD, ip, sp valid, yielded value in *sp
|
|
|
|
// MP_VM_RETURN_EXCEPTION, exception in fastn[0]
|
2014-05-31 09:50:46 -04:00
|
|
|
mp_vm_return_kind_t mp_execute_bytecode2(mp_code_state *code_state, volatile mp_obj_t inject_exc) {
|
2014-05-21 15:32:59 -04:00
|
|
|
#if MICROPY_OPT_COMPUTED_GOTO
|
2014-04-15 09:29:08 -04:00
|
|
|
#include "vmentrytable.h"
|
2014-04-15 03:57:01 -04:00
|
|
|
#define DISPATCH() do { \
|
2014-04-22 20:40:24 -04:00
|
|
|
TRACE(ip); \
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
code_state->ip = ip; \
|
2014-04-27 13:19:06 -04:00
|
|
|
goto *entry_table[*ip++]; \
|
2014-04-14 11:22:44 -04:00
|
|
|
} while(0)
|
2014-04-15 03:57:01 -04:00
|
|
|
#define ENTRY(op) entry_##op
|
|
|
|
#define ENTRY_DEFAULT entry_default
|
2014-04-14 11:22:44 -04:00
|
|
|
#else
|
2014-04-15 03:57:01 -04:00
|
|
|
#define DISPATCH() break
|
|
|
|
#define ENTRY(op) case op
|
|
|
|
#define ENTRY_DEFAULT default
|
2014-04-14 11:22:44 -04:00
|
|
|
#endif
|
|
|
|
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
// nlr_raise needs to be implemented as a goto, so that the C compiler's flow analyser
|
|
|
|
// sees that it's possible for us to jump from the dispatch loop to the exception
|
|
|
|
// handler. Without this, the code may have a different stack layout in the dispatch
|
|
|
|
// loop and the exception handler, leading to very obscure bugs.
|
|
|
|
#define RAISE(o) do { nlr_pop(); nlr.ret_val = o; goto exception_handler; } while(0)
|
2013-10-04 14:53:11 -04:00
|
|
|
|
2014-05-31 09:50:46 -04:00
|
|
|
// Pointers which are constant for particular invocation of mp_execute_bytecode2()
|
|
|
|
mp_obj_t *const fastn = &code_state->state[code_state->n_state - 1];
|
|
|
|
mp_exc_stack_t *const exc_stack = (mp_exc_stack_t*)(code_state->state + code_state->n_state);
|
|
|
|
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
// variables that are visible to the exception handler (declared volatile)
|
2014-05-31 09:50:46 -04:00
|
|
|
volatile bool currently_in_except_block = MP_TAGPTR_TAG(code_state->exc_sp); // 0 or 1, to detect nested exceptions
|
|
|
|
mp_exc_stack_t *volatile exc_sp = MP_TAGPTR_PTR(code_state->exc_sp); // stack grows up, exc_sp points to top of stack
|
2013-10-15 18:46:01 -04:00
|
|
|
|
2013-10-15 17:25:17 -04:00
|
|
|
// outer exception handling loop
|
2013-10-04 14:53:11 -04:00
|
|
|
for (;;) {
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
nlr_buf_t nlr;
|
2014-03-26 14:37:06 -04:00
|
|
|
outer_dispatch_loop:
|
2013-10-15 17:25:17 -04:00
|
|
|
if (nlr_push(&nlr) == 0) {
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
// local variables that are not visible to the exception handler
|
2014-05-31 09:50:46 -04:00
|
|
|
const byte *ip = code_state->ip;
|
|
|
|
mp_obj_t *sp = code_state->sp;
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
machine_uint_t unum;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t obj_shared;
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
|
2014-03-22 11:50:12 -04:00
|
|
|
// If we have exception to inject, now that we finish setting up
|
|
|
|
// execution context, raise it. This works as if RAISE_VARARGS
|
|
|
|
// bytecode was executed.
|
2014-03-26 11:36:12 -04:00
|
|
|
// Injecting exc into yield from generator is a special case,
|
|
|
|
// handled by MP_BC_YIELD_FROM itself
|
|
|
|
if (inject_exc != MP_OBJ_NULL && *ip != MP_BC_YIELD_FROM) {
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t exc = inject_exc;
|
2014-03-22 11:50:12 -04:00
|
|
|
inject_exc = MP_OBJ_NULL;
|
2014-05-25 17:58:04 -04:00
|
|
|
exc = mp_make_raise_obj(exc);
|
|
|
|
RAISE(exc);
|
2014-03-22 11:50:12 -04:00
|
|
|
}
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
|
2013-10-15 17:25:17 -04:00
|
|
|
// loop to execute byte code
|
|
|
|
for (;;) {
|
2014-01-31 17:55:05 -05:00
|
|
|
dispatch_loop:
|
2014-05-21 15:32:59 -04:00
|
|
|
#if MICROPY_OPT_COMPUTED_GOTO
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
|
|
|
#else
|
2014-04-22 20:40:24 -04:00
|
|
|
TRACE(ip);
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
code_state->ip = ip;
|
2014-04-27 13:19:06 -04:00
|
|
|
switch (*ip++) {
|
2014-04-14 11:22:44 -04:00
|
|
|
#endif
|
2014-04-09 10:26:46 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_FALSE):
|
|
|
|
PUSH(mp_const_false);
|
|
|
|
DISPATCH();
|
2014-04-09 10:26:46 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_NONE):
|
|
|
|
PUSH(mp_const_none);
|
|
|
|
DISPATCH();
|
2014-03-23 15:19:02 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_TRUE):
|
|
|
|
PUSH(mp_const_true);
|
|
|
|
DISPATCH();
|
2014-04-08 16:11:49 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_ELLIPSIS):
|
|
|
|
PUSH((mp_obj_t)&mp_const_ellipsis_obj);
|
|
|
|
DISPATCH();
|
2013-10-15 17:25:17 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_SMALL_INT): {
|
|
|
|
machine_int_t num = 0;
|
|
|
|
if ((ip[0] & 0x40) != 0) {
|
|
|
|
// Number is negative
|
|
|
|
num--;
|
|
|
|
}
|
|
|
|
do {
|
|
|
|
num = (num << 7) | (*ip & 0x7f);
|
|
|
|
} while ((*ip++ & 0x80) != 0);
|
|
|
|
PUSH(MP_OBJ_NEW_SMALL_INT(num));
|
|
|
|
DISPATCH();
|
|
|
|
}
|
2013-10-15 17:25:17 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_INT): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
2014-05-28 09:07:21 -04:00
|
|
|
PUSH(mp_load_const_int(qst));
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_DEC): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
PUSH(mp_load_const_dec(qst));
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_BYTES): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
PUSH(mp_load_const_bytes(qst));
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_CONST_STRING): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
PUSH(mp_load_const_str(qst));
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_NULL):
|
|
|
|
PUSH(MP_OBJ_NULL);
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_FAST_0):
|
2014-05-25 17:58:04 -04:00
|
|
|
obj_shared = fastn[0];
|
2014-04-14 11:22:44 -04:00
|
|
|
goto load_check;
|
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_FAST_1):
|
2014-05-25 17:58:04 -04:00
|
|
|
obj_shared = fastn[-1];
|
2014-04-14 11:22:44 -04:00
|
|
|
goto load_check;
|
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_FAST_2):
|
2014-05-25 17:58:04 -04:00
|
|
|
obj_shared = fastn[-2];
|
2014-04-14 11:22:44 -04:00
|
|
|
goto load_check;
|
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_FAST_N):
|
|
|
|
DECODE_UINT;
|
2014-05-25 17:58:04 -04:00
|
|
|
obj_shared = fastn[-unum];
|
2014-04-14 11:22:44 -04:00
|
|
|
load_check:
|
2014-05-25 17:58:04 -04:00
|
|
|
if (obj_shared == MP_OBJ_NULL) {
|
|
|
|
local_name_error: {
|
|
|
|
mp_obj_t obj = mp_obj_new_exception_msg(&mp_type_NameError, "local variable referenced before assignment");
|
|
|
|
RAISE(obj);
|
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
2014-05-25 17:58:04 -04:00
|
|
|
PUSH(obj_shared);
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_DEREF):
|
|
|
|
DECODE_UINT;
|
2014-05-25 17:58:04 -04:00
|
|
|
obj_shared = mp_obj_cell_get(fastn[-unum]);
|
2014-04-14 11:22:44 -04:00
|
|
|
goto load_check;
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_NAME): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
PUSH(mp_load_name(qst));
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_GLOBAL): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
PUSH(mp_load_global(qst));
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_ATTR): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
SET_TOP(mp_load_attr(TOP(), qst));
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_METHOD): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
mp_load_method(*sp, qst, sp);
|
|
|
|
sp += 1;
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_LOAD_BUILD_CLASS):
|
|
|
|
PUSH(mp_load_build_class());
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_LOAD_SUBSCR): {
|
|
|
|
mp_obj_t index = POP();
|
|
|
|
SET_TOP(mp_obj_subscr(TOP(), index, MP_OBJ_SENTINEL));
|
2014-04-17 17:10:53 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-17 17:10:53 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_STORE_FAST_0):
|
|
|
|
fastn[0] = POP();
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_STORE_FAST_1):
|
|
|
|
fastn[-1] = POP();
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_STORE_FAST_2):
|
|
|
|
fastn[-2] = POP();
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_STORE_FAST_N):
|
|
|
|
DECODE_UINT;
|
|
|
|
fastn[-unum] = POP();
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_STORE_DEREF):
|
|
|
|
DECODE_UINT;
|
|
|
|
mp_obj_cell_set(fastn[-unum], POP());
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_STORE_NAME): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
mp_store_name(qst, POP());
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_STORE_GLOBAL): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
mp_store_global(qst, POP());
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_STORE_ATTR): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
mp_store_attr(sp[0], qst, sp[-1]);
|
|
|
|
sp -= 2;
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_STORE_SUBSCR):
|
2014-04-17 17:10:53 -04:00
|
|
|
mp_obj_subscr(sp[-1], sp[0], sp[-2]);
|
2014-04-14 11:22:44 -04:00
|
|
|
sp -= 3;
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_DELETE_FAST):
|
|
|
|
DECODE_UINT;
|
|
|
|
if (fastn[-unum] == MP_OBJ_NULL) {
|
|
|
|
goto local_name_error;
|
|
|
|
}
|
|
|
|
fastn[-unum] = MP_OBJ_NULL;
|
|
|
|
DISPATCH();
|
2013-10-15 17:25:17 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_DELETE_DEREF):
|
|
|
|
DECODE_UINT;
|
|
|
|
if (mp_obj_cell_get(fastn[-unum]) == MP_OBJ_NULL) {
|
|
|
|
goto local_name_error;
|
|
|
|
}
|
|
|
|
mp_obj_cell_set(fastn[-unum], MP_OBJ_NULL);
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_DELETE_NAME): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
mp_delete_name(qst);
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_DELETE_GLOBAL): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
|
|
|
mp_delete_global(qst);
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_DUP_TOP): {
|
|
|
|
mp_obj_t top = TOP();
|
|
|
|
PUSH(top);
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_DUP_TOP_TWO):
|
|
|
|
sp += 2;
|
|
|
|
sp[0] = sp[-2];
|
|
|
|
sp[-1] = sp[-3];
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_POP_TOP):
|
|
|
|
sp -= 1;
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_ROT_TWO): {
|
|
|
|
mp_obj_t top = sp[0];
|
2014-04-14 11:22:44 -04:00
|
|
|
sp[0] = sp[-1];
|
2014-05-25 17:58:04 -04:00
|
|
|
sp[-1] = top;
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_ROT_THREE): {
|
|
|
|
mp_obj_t top = sp[0];
|
2014-04-14 11:22:44 -04:00
|
|
|
sp[0] = sp[-1];
|
|
|
|
sp[-1] = sp[-2];
|
2014-05-25 17:58:04 -04:00
|
|
|
sp[-2] = top;
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_JUMP):
|
|
|
|
DECODE_SLABEL;
|
|
|
|
ip += unum;
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_POP_JUMP_IF_TRUE):
|
|
|
|
DECODE_SLABEL;
|
|
|
|
if (mp_obj_is_true(POP())) {
|
|
|
|
ip += unum;
|
|
|
|
}
|
|
|
|
DISPATCH();
|
2013-11-09 15:12:32 -05:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_POP_JUMP_IF_FALSE):
|
|
|
|
DECODE_SLABEL;
|
|
|
|
if (!mp_obj_is_true(POP())) {
|
|
|
|
ip += unum;
|
|
|
|
}
|
|
|
|
DISPATCH();
|
2013-11-09 15:12:32 -05:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_JUMP_IF_TRUE_OR_POP):
|
|
|
|
DECODE_SLABEL;
|
|
|
|
if (mp_obj_is_true(TOP())) {
|
|
|
|
ip += unum;
|
|
|
|
} else {
|
|
|
|
sp--;
|
|
|
|
}
|
|
|
|
DISPATCH();
|
2013-10-15 17:25:17 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_JUMP_IF_FALSE_OR_POP):
|
|
|
|
DECODE_SLABEL;
|
|
|
|
if (mp_obj_is_true(TOP())) {
|
|
|
|
sp--;
|
|
|
|
} else {
|
|
|
|
ip += unum;
|
|
|
|
}
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_SETUP_WITH): {
|
|
|
|
mp_obj_t obj = TOP();
|
|
|
|
SET_TOP(mp_load_attr(obj, MP_QSTR___exit__));
|
|
|
|
mp_load_method(obj, MP_QSTR___enter__, sp + 1);
|
|
|
|
mp_obj_t ret = mp_call_method_n_kw(0, 0, sp + 1);
|
2014-04-14 11:22:44 -04:00
|
|
|
PUSH_EXC_BLOCK();
|
2014-05-25 17:58:04 -04:00
|
|
|
PUSH(ret);
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_WITH_CLEANUP): {
|
|
|
|
// Arriving here, there's "exception control block" on top of stack,
|
|
|
|
// and __exit__ bound method underneath it. Bytecode calls __exit__,
|
|
|
|
// and "deletes" it off stack, shifting "exception control block"
|
|
|
|
// to its place.
|
|
|
|
static const mp_obj_t no_exc[] = {mp_const_none, mp_const_none, mp_const_none};
|
|
|
|
if (TOP() == mp_const_none) {
|
|
|
|
sp--;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t obj = TOP();
|
2014-04-14 11:22:44 -04:00
|
|
|
SET_TOP(mp_const_none);
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_call_function_n_kw(obj, 3, 0, no_exc);
|
2014-04-14 11:22:44 -04:00
|
|
|
} else if (MP_OBJ_IS_SMALL_INT(TOP())) {
|
|
|
|
mp_obj_t cause = POP();
|
|
|
|
switch (MP_OBJ_SMALL_INT_VALUE(cause)) {
|
|
|
|
case UNWIND_RETURN: {
|
|
|
|
mp_obj_t retval = POP();
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_call_function_n_kw(TOP(), 3, 0, no_exc);
|
2014-04-14 11:22:44 -04:00
|
|
|
SET_TOP(retval);
|
|
|
|
PUSH(cause);
|
|
|
|
break;
|
2014-03-28 22:10:11 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
case UNWIND_JUMP: {
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_call_function_n_kw(sp[-2], 3, 0, no_exc);
|
2014-04-14 11:22:44 -04:00
|
|
|
// Pop __exit__ boundmethod at sp[-2]
|
|
|
|
sp[-2] = sp[-1];
|
|
|
|
sp[-1] = sp[0];
|
|
|
|
SET_TOP(cause);
|
|
|
|
break;
|
2014-03-28 22:10:11 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
default:
|
|
|
|
assert(0);
|
2014-03-28 22:10:11 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
} else if (mp_obj_is_exception_type(TOP())) {
|
|
|
|
mp_obj_t args[3] = {sp[0], sp[-1], sp[-2]};
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t ret_value = mp_call_function_n_kw(sp[-3], 3, 0, args);
|
2014-04-14 11:22:44 -04:00
|
|
|
// Pop __exit__ boundmethod at sp[-3]
|
2014-05-25 17:58:04 -04:00
|
|
|
// TODO: Once semantics is proven, optimize for case when ret_value == True
|
2014-04-14 11:22:44 -04:00
|
|
|
sp[-3] = sp[-2];
|
|
|
|
sp[-2] = sp[-1];
|
|
|
|
sp[-1] = sp[0];
|
|
|
|
sp--;
|
2014-05-25 17:58:04 -04:00
|
|
|
if (mp_obj_is_true(ret_value)) {
|
2014-04-14 11:22:44 -04:00
|
|
|
// This is what CPython does
|
|
|
|
//PUSH(MP_OBJ_NEW_SMALL_INT(UNWIND_SILENCED));
|
|
|
|
// But what we need to do is - pop exception from value stack...
|
|
|
|
sp -= 3;
|
|
|
|
// ... pop "with" exception handler, and signal END_FINALLY
|
|
|
|
// to just execute finally handler normally (by pushing None
|
|
|
|
// on value stack)
|
2014-02-01 15:08:18 -05:00
|
|
|
assert(exc_sp >= exc_stack);
|
2014-04-14 11:22:44 -04:00
|
|
|
assert(exc_sp->opcode == MP_BC_SETUP_WITH);
|
|
|
|
POP_EXC_BLOCK();
|
|
|
|
PUSH(mp_const_none);
|
2014-01-30 06:49:18 -05:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
} else {
|
|
|
|
assert(0);
|
|
|
|
}
|
|
|
|
DISPATCH();
|
|
|
|
}
|
2013-10-15 17:25:17 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_UNWIND_JUMP):
|
|
|
|
DECODE_SLABEL;
|
|
|
|
PUSH((void*)(ip + unum)); // push destination ip for jump
|
2014-05-30 10:20:41 -04:00
|
|
|
PUSH((void*)(machine_uint_t)(*ip)); // push number of exception handlers to unwind (0x80 bit set if we also need to pop stack)
|
2014-04-14 11:22:44 -04:00
|
|
|
unwind_jump:
|
|
|
|
unum = (machine_uint_t)POP(); // get number of exception handlers to unwind
|
2014-05-30 10:20:41 -04:00
|
|
|
while ((unum & 0x7f) > 0) {
|
2014-04-14 11:22:44 -04:00
|
|
|
unum -= 1;
|
2014-03-22 07:49:31 -04:00
|
|
|
assert(exc_sp >= exc_stack);
|
2014-04-14 11:22:44 -04:00
|
|
|
if (exc_sp->opcode == MP_BC_SETUP_FINALLY || exc_sp->opcode == MP_BC_SETUP_WITH) {
|
|
|
|
// We're going to run "finally" code as a coroutine
|
|
|
|
// (not calling it recursively). Set up a sentinel
|
|
|
|
// on a stack so it can return back to us when it is
|
|
|
|
// done (when END_FINALLY reached).
|
|
|
|
PUSH((void*)unum); // push number of exception handlers left to unwind
|
|
|
|
PUSH(MP_OBJ_NEW_SMALL_INT(UNWIND_JUMP)); // push sentinel
|
|
|
|
ip = exc_sp->handler; // get exception handler byte code address
|
|
|
|
exc_sp--; // pop exception handler
|
|
|
|
goto dispatch_loop; // run the exception handler
|
2014-02-01 18:04:09 -05:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
exc_sp--;
|
|
|
|
}
|
|
|
|
ip = (const byte*)POP(); // pop destination ip for jump
|
2014-05-30 10:20:41 -04:00
|
|
|
if (unum != 0) {
|
|
|
|
sp--;
|
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
// matched against: POP_BLOCK or POP_EXCEPT (anything else?)
|
|
|
|
ENTRY(MP_BC_SETUP_EXCEPT):
|
|
|
|
ENTRY(MP_BC_SETUP_FINALLY):
|
|
|
|
PUSH_EXC_BLOCK();
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_END_FINALLY):
|
|
|
|
// not fully implemented
|
|
|
|
// if TOS is an exception, reraises the exception (3 values on TOS)
|
|
|
|
// if TOS is None, just pops it and continues
|
|
|
|
// if TOS is an integer, does something else
|
|
|
|
// else error
|
|
|
|
if (mp_obj_is_exception_type(TOP())) {
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
RAISE(sp[-1]);
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
|
|
|
if (TOP() == mp_const_none) {
|
2014-01-18 09:10:48 -05:00
|
|
|
sp--;
|
2014-04-14 11:22:44 -04:00
|
|
|
} else if (MP_OBJ_IS_SMALL_INT(TOP())) {
|
|
|
|
// We finished "finally" coroutine and now dispatch back
|
|
|
|
// to our caller, based on TOS value
|
|
|
|
mp_unwind_reason_t reason = MP_OBJ_SMALL_INT_VALUE(POP());
|
|
|
|
switch (reason) {
|
|
|
|
case UNWIND_RETURN:
|
|
|
|
goto unwind_return;
|
|
|
|
case UNWIND_JUMP:
|
|
|
|
goto unwind_jump;
|
|
|
|
}
|
|
|
|
assert(0);
|
|
|
|
} else {
|
|
|
|
assert(0);
|
|
|
|
}
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_GET_ITER):
|
|
|
|
SET_TOP(mp_getiter(TOP()));
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_FOR_ITER): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_ULABEL; // the jump offset if iteration finishes; for labels are always forward
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
code_state->sp = sp;
|
2014-05-11 13:32:39 -04:00
|
|
|
assert(TOP());
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t value = mp_iternext_allow_raise(TOP());
|
|
|
|
if (value == MP_OBJ_STOP_ITERATION) {
|
2014-04-14 11:22:44 -04:00
|
|
|
--sp; // pop the exhausted iterator
|
|
|
|
ip += unum; // jump to after for-block
|
|
|
|
} else {
|
2014-05-25 17:58:04 -04:00
|
|
|
PUSH(value); // push the next iteration value
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
// matched against: SETUP_EXCEPT, SETUP_FINALLY, SETUP_WITH
|
|
|
|
ENTRY(MP_BC_POP_BLOCK):
|
|
|
|
// we are exiting an exception handler, so pop the last one of the exception-stack
|
|
|
|
assert(exc_sp >= exc_stack);
|
|
|
|
POP_EXC_BLOCK();
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
// matched against: SETUP_EXCEPT
|
|
|
|
ENTRY(MP_BC_POP_EXCEPT):
|
|
|
|
// TODO need to work out how blocks work etc
|
|
|
|
// pops block, checks it's an exception block, and restores the stack, saving the 3 exception values to local threadstate
|
|
|
|
assert(exc_sp >= exc_stack);
|
|
|
|
assert(currently_in_except_block);
|
|
|
|
//sp = (mp_obj_t*)(*exc_sp--);
|
|
|
|
//exc_sp--; // discard ip
|
|
|
|
POP_EXC_BLOCK();
|
|
|
|
//sp -= 3; // pop 3 exception values
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_NOT):
|
|
|
|
if (TOP() == mp_const_true) {
|
|
|
|
SET_TOP(mp_const_false);
|
|
|
|
} else {
|
|
|
|
SET_TOP(mp_const_true);
|
|
|
|
}
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_UNARY_OP):
|
|
|
|
unum = *ip++;
|
|
|
|
SET_TOP(mp_unary_op(unum, TOP()));
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_BINARY_OP): {
|
2014-04-14 11:22:44 -04:00
|
|
|
unum = *ip++;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t rhs = POP();
|
|
|
|
mp_obj_t lhs = TOP();
|
|
|
|
SET_TOP(mp_binary_op(unum, lhs, rhs));
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_BUILD_TUPLE):
|
|
|
|
DECODE_UINT;
|
|
|
|
sp -= unum - 1;
|
|
|
|
SET_TOP(mp_obj_new_tuple(unum, sp));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_BUILD_LIST):
|
|
|
|
DECODE_UINT;
|
|
|
|
sp -= unum - 1;
|
|
|
|
SET_TOP(mp_obj_new_list(unum, sp));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_LIST_APPEND):
|
|
|
|
DECODE_UINT;
|
|
|
|
// I think it's guaranteed by the compiler that sp[unum] is a list
|
|
|
|
mp_obj_list_append(sp[-unum], sp[0]);
|
|
|
|
sp--;
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_BUILD_MAP):
|
|
|
|
DECODE_UINT;
|
|
|
|
PUSH(mp_obj_new_dict(unum));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_STORE_MAP):
|
|
|
|
sp -= 2;
|
|
|
|
mp_obj_dict_store(sp[0], sp[2], sp[1]);
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_MAP_ADD):
|
|
|
|
DECODE_UINT;
|
|
|
|
// I think it's guaranteed by the compiler that sp[-unum - 1] is a map
|
|
|
|
mp_obj_dict_store(sp[-unum - 1], sp[0], sp[-1]);
|
|
|
|
sp -= 2;
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_BUILD_SET):
|
|
|
|
DECODE_UINT;
|
|
|
|
sp -= unum - 1;
|
|
|
|
SET_TOP(mp_obj_new_set(unum, sp));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_SET_ADD):
|
|
|
|
DECODE_UINT;
|
|
|
|
// I think it's guaranteed by the compiler that sp[-unum] is a set
|
|
|
|
mp_obj_set_store(sp[-unum], sp[0]);
|
|
|
|
sp--;
|
|
|
|
DISPATCH();
|
2013-10-16 15:57:49 -04:00
|
|
|
|
2014-06-01 08:32:54 -04:00
|
|
|
#if MICROPY_PY_BUILTINS_SLICE
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_BUILD_SLICE):
|
|
|
|
DECODE_UINT;
|
|
|
|
if (unum == 2) {
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t stop = POP();
|
|
|
|
mp_obj_t start = TOP();
|
|
|
|
SET_TOP(mp_obj_new_slice(start, stop, mp_const_none));
|
2014-04-14 11:22:44 -04:00
|
|
|
} else {
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t step = POP();
|
|
|
|
mp_obj_t stop = POP();
|
|
|
|
mp_obj_t start = TOP();
|
|
|
|
SET_TOP(mp_obj_new_slice(start, stop, step));
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
|
|
|
DISPATCH();
|
2014-01-03 18:34:23 -05:00
|
|
|
#endif
|
2014-01-02 19:48:56 -05:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_UNPACK_SEQUENCE):
|
|
|
|
DECODE_UINT;
|
|
|
|
mp_unpack_sequence(sp[0], unum, sp);
|
|
|
|
sp += unum - 1;
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_UNPACK_EX):
|
|
|
|
DECODE_UINT;
|
|
|
|
mp_unpack_ex(sp[0], unum, sp);
|
|
|
|
sp += (unum & 0xff) + ((unum >> 8) & 0xff);
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_MAKE_FUNCTION):
|
|
|
|
DECODE_PTR;
|
|
|
|
PUSH(mp_make_function_from_raw_code((mp_raw_code_t*)unum, MP_OBJ_NULL, MP_OBJ_NULL));
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_MAKE_FUNCTION_DEFARGS): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_PTR;
|
|
|
|
// Stack layout: def_tuple def_dict <- TOS
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t def_dict = POP();
|
|
|
|
SET_TOP(mp_make_function_from_raw_code((mp_raw_code_t*)unum, TOP(), def_dict));
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-04-20 12:50:40 -04:00
|
|
|
ENTRY(MP_BC_MAKE_CLOSURE): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_PTR;
|
2014-04-20 12:50:40 -04:00
|
|
|
machine_uint_t n_closed_over = *ip++;
|
|
|
|
// Stack layout: closed_overs <- TOS
|
|
|
|
sp -= n_closed_over - 1;
|
|
|
|
SET_TOP(mp_make_closure_from_raw_code((mp_raw_code_t*)unum, n_closed_over, sp));
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-04-20 12:50:40 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-04-20 12:50:40 -04:00
|
|
|
ENTRY(MP_BC_MAKE_CLOSURE_DEFARGS): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_PTR;
|
2014-04-20 12:50:40 -04:00
|
|
|
machine_uint_t n_closed_over = *ip++;
|
|
|
|
// Stack layout: def_tuple def_dict closed_overs <- TOS
|
|
|
|
sp -= 2 + n_closed_over - 1;
|
|
|
|
SET_TOP(mp_make_closure_from_raw_code((mp_raw_code_t*)unum, 0x100 | n_closed_over, sp));
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-04-20 12:50:40 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_CALL_FUNCTION):
|
|
|
|
DECODE_UINT;
|
|
|
|
// unum & 0xff == n_positional
|
|
|
|
// (unum >> 8) & 0xff == n_keyword
|
|
|
|
sp -= (unum & 0xff) + ((unum >> 7) & 0x1fe);
|
|
|
|
SET_TOP(mp_call_function_n_kw(*sp, unum & 0xff, (unum >> 8) & 0xff, sp + 1));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_CALL_FUNCTION_VAR_KW):
|
|
|
|
DECODE_UINT;
|
|
|
|
// unum & 0xff == n_positional
|
|
|
|
// (unum >> 8) & 0xff == n_keyword
|
|
|
|
// We have folowing stack layout here:
|
|
|
|
// fun arg0 arg1 ... kw0 val0 kw1 val1 ... seq dict <- TOS
|
|
|
|
sp -= (unum & 0xff) + ((unum >> 7) & 0x1fe) + 2;
|
|
|
|
SET_TOP(mp_call_method_n_kw_var(false, unum, sp));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_CALL_METHOD):
|
|
|
|
DECODE_UINT;
|
|
|
|
// unum & 0xff == n_positional
|
|
|
|
// (unum >> 8) & 0xff == n_keyword
|
|
|
|
sp -= (unum & 0xff) + ((unum >> 7) & 0x1fe) + 1;
|
|
|
|
SET_TOP(mp_call_method_n_kw(unum & 0xff, (unum >> 8) & 0xff, sp));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_CALL_METHOD_VAR_KW):
|
|
|
|
DECODE_UINT;
|
|
|
|
// unum & 0xff == n_positional
|
|
|
|
// (unum >> 8) & 0xff == n_keyword
|
|
|
|
// We have folowing stack layout here:
|
|
|
|
// fun self arg0 arg1 ... kw0 val0 kw1 val1 ... seq dict <- TOS
|
|
|
|
sp -= (unum & 0xff) + ((unum >> 7) & 0x1fe) + 3;
|
|
|
|
SET_TOP(mp_call_method_n_kw_var(true, unum, sp));
|
|
|
|
DISPATCH();
|
|
|
|
|
|
|
|
ENTRY(MP_BC_RETURN_VALUE):
|
2014-01-31 17:55:05 -05:00
|
|
|
unwind_return:
|
2014-04-14 11:22:44 -04:00
|
|
|
while (exc_sp >= exc_stack) {
|
|
|
|
if (exc_sp->opcode == MP_BC_SETUP_FINALLY || exc_sp->opcode == MP_BC_SETUP_WITH) {
|
|
|
|
// We're going to run "finally" code as a coroutine
|
|
|
|
// (not calling it recursively). Set up a sentinel
|
|
|
|
// on a stack so it can return back to us when it is
|
|
|
|
// done (when END_FINALLY reached).
|
|
|
|
PUSH(MP_OBJ_NEW_SMALL_INT(UNWIND_RETURN));
|
|
|
|
ip = exc_sp->handler;
|
|
|
|
// We don't need to do anything with sp, finally is just
|
|
|
|
// syntactic sugar for sequential execution??
|
|
|
|
// sp =
|
2014-01-31 17:55:05 -05:00
|
|
|
exc_sp--;
|
2014-04-14 11:22:44 -04:00
|
|
|
goto dispatch_loop;
|
2014-01-31 17:55:05 -05:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
exc_sp--;
|
|
|
|
}
|
|
|
|
nlr_pop();
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->sp = sp;
|
2014-04-14 11:22:44 -04:00
|
|
|
assert(exc_sp == exc_stack - 1);
|
|
|
|
return MP_VM_RETURN_NORMAL;
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_RAISE_VARARGS): {
|
2014-04-14 11:22:44 -04:00
|
|
|
unum = *ip++;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t obj;
|
2014-04-14 11:22:44 -04:00
|
|
|
assert(unum <= 1);
|
|
|
|
if (unum == 0) {
|
|
|
|
// search for the inner-most previous exception, to reraise it
|
2014-05-25 17:58:04 -04:00
|
|
|
obj = MP_OBJ_NULL;
|
2014-04-14 11:22:44 -04:00
|
|
|
for (mp_exc_stack_t *e = exc_sp; e >= exc_stack; e--) {
|
|
|
|
if (e->prev_exc != MP_OBJ_NULL) {
|
2014-05-25 17:58:04 -04:00
|
|
|
obj = e->prev_exc;
|
2014-04-14 11:22:44 -04:00
|
|
|
break;
|
2014-03-29 13:44:15 -04:00
|
|
|
}
|
2014-03-26 08:42:17 -04:00
|
|
|
}
|
2014-05-25 17:58:04 -04:00
|
|
|
if (obj == MP_OBJ_NULL) {
|
|
|
|
obj = mp_obj_new_exception_msg(&mp_type_RuntimeError, "No active exception to reraise");
|
|
|
|
RAISE(obj);
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
|
|
|
} else {
|
2014-05-25 17:58:04 -04:00
|
|
|
obj = POP();
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
2014-05-25 17:58:04 -04:00
|
|
|
obj = mp_make_raise_obj(obj);
|
|
|
|
RAISE(obj);
|
|
|
|
}
|
2014-01-10 09:09:55 -05:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_YIELD_VALUE):
|
2014-03-26 11:36:12 -04:00
|
|
|
yield:
|
2014-04-14 11:22:44 -04:00
|
|
|
nlr_pop();
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->ip = ip;
|
|
|
|
code_state->sp = sp;
|
|
|
|
code_state->exc_sp = MP_TAGPTR_MAKE(exc_sp, currently_in_except_block);
|
2014-04-14 11:22:44 -04:00
|
|
|
return MP_VM_RETURN_YIELD;
|
2013-10-15 17:25:17 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
ENTRY(MP_BC_YIELD_FROM): {
|
2014-03-26 11:36:12 -04:00
|
|
|
//#define EXC_MATCH(exc, type) MP_OBJ_IS_TYPE(exc, type)
|
|
|
|
#define EXC_MATCH(exc, type) mp_obj_exception_match(exc, type)
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
#define GENERATOR_EXIT_IF_NEEDED(t) if (t != MP_OBJ_NULL && EXC_MATCH(t, &mp_type_GeneratorExit)) { RAISE(t); }
|
2014-04-14 11:22:44 -04:00
|
|
|
mp_vm_return_kind_t ret_kind;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t send_value = POP();
|
2014-04-14 11:22:44 -04:00
|
|
|
mp_obj_t t_exc = MP_OBJ_NULL;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t ret_value;
|
2014-04-14 11:22:44 -04:00
|
|
|
if (inject_exc != MP_OBJ_NULL) {
|
|
|
|
t_exc = inject_exc;
|
|
|
|
inject_exc = MP_OBJ_NULL;
|
2014-05-25 17:58:04 -04:00
|
|
|
ret_kind = mp_resume(TOP(), MP_OBJ_NULL, t_exc, &ret_value);
|
2014-04-14 11:22:44 -04:00
|
|
|
} else {
|
2014-05-25 17:58:04 -04:00
|
|
|
ret_kind = mp_resume(TOP(), send_value, MP_OBJ_NULL, &ret_value);
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
2014-03-26 11:36:12 -04:00
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
if (ret_kind == MP_VM_RETURN_YIELD) {
|
|
|
|
ip--;
|
2014-05-25 17:58:04 -04:00
|
|
|
PUSH(ret_value);
|
2014-04-14 11:22:44 -04:00
|
|
|
goto yield;
|
|
|
|
}
|
|
|
|
if (ret_kind == MP_VM_RETURN_NORMAL) {
|
|
|
|
// Pop exhausted gen
|
|
|
|
sp--;
|
2014-05-25 17:58:04 -04:00
|
|
|
if (ret_value == MP_OBJ_NULL) {
|
2014-04-14 11:22:44 -04:00
|
|
|
// Optimize StopIteration
|
|
|
|
// TODO: get StopIteration's value
|
|
|
|
PUSH(mp_const_none);
|
|
|
|
} else {
|
2014-05-25 17:58:04 -04:00
|
|
|
PUSH(ret_value);
|
2014-03-26 11:36:12 -04:00
|
|
|
}
|
|
|
|
|
2014-04-14 11:22:44 -04:00
|
|
|
// If we injected GeneratorExit downstream, then even
|
|
|
|
// if it was swallowed, we re-raise GeneratorExit
|
|
|
|
GENERATOR_EXIT_IF_NEEDED(t_exc);
|
|
|
|
DISPATCH();
|
|
|
|
}
|
|
|
|
if (ret_kind == MP_VM_RETURN_EXCEPTION) {
|
|
|
|
// Pop exhausted gen
|
|
|
|
sp--;
|
2014-05-25 17:58:04 -04:00
|
|
|
if (EXC_MATCH(ret_value, &mp_type_StopIteration)) {
|
|
|
|
PUSH(mp_obj_exception_get_value(ret_value));
|
2014-03-26 13:24:03 -04:00
|
|
|
// If we injected GeneratorExit downstream, then even
|
|
|
|
// if it was swallowed, we re-raise GeneratorExit
|
|
|
|
GENERATOR_EXIT_IF_NEEDED(t_exc);
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
|
|
|
} else {
|
2014-05-25 17:58:04 -04:00
|
|
|
RAISE(ret_value);
|
2014-03-26 11:36:12 -04:00
|
|
|
}
|
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
}
|
2014-03-26 11:36:12 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_IMPORT_NAME): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t obj = POP();
|
|
|
|
SET_TOP(mp_import_name(qst, obj, TOP()));
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY(MP_BC_IMPORT_FROM): {
|
2014-04-14 11:22:44 -04:00
|
|
|
DECODE_QSTR;
|
2014-05-25 17:58:04 -04:00
|
|
|
mp_obj_t obj = mp_import_from(TOP(), qst);
|
|
|
|
PUSH(obj);
|
2014-04-14 11:22:44 -04:00
|
|
|
DISPATCH();
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
2014-04-14 11:22:44 -04:00
|
|
|
|
|
|
|
ENTRY(MP_BC_IMPORT_STAR):
|
|
|
|
mp_import_all(POP());
|
|
|
|
DISPATCH();
|
|
|
|
|
2014-05-25 17:58:04 -04:00
|
|
|
ENTRY_DEFAULT: {
|
|
|
|
mp_obj_t obj = mp_obj_new_exception_msg(&mp_type_NotImplementedError, "byte code not implemented");
|
2014-04-14 11:22:44 -04:00
|
|
|
nlr_pop();
|
2014-05-25 17:58:04 -04:00
|
|
|
fastn[0] = obj;
|
2014-04-14 11:22:44 -04:00
|
|
|
return MP_VM_RETURN_EXCEPTION;
|
2014-05-25 17:58:04 -04:00
|
|
|
}
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
|
2014-05-21 15:32:59 -04:00
|
|
|
#if !MICROPY_OPT_COMPUTED_GOTO
|
2014-04-15 03:57:01 -04:00
|
|
|
} // switch
|
2014-04-14 11:22:44 -04:00
|
|
|
#endif
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
} // for loop
|
2013-10-15 17:25:17 -04:00
|
|
|
|
|
|
|
} else {
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
exception_handler:
|
2013-10-15 17:25:17 -04:00
|
|
|
// exception occurred
|
|
|
|
|
2014-03-26 14:37:06 -04:00
|
|
|
// check if it's a StopIteration within a for block
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
if (*code_state->ip == MP_BC_FOR_ITER && mp_obj_is_subclass_fast(mp_obj_get_type(nlr.ret_val), &mp_type_StopIteration)) {
|
|
|
|
const byte *ip = code_state->ip + 1;
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
machine_uint_t unum;
|
2014-03-26 14:37:06 -04:00
|
|
|
DECODE_ULABEL; // the jump offset if iteration finishes; for labels are always forward
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->ip = ip + unum; // jump to after for-block
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
code_state->sp -= 1; // pop the exhausted iterator
|
2014-03-26 14:37:06 -04:00
|
|
|
goto outer_dispatch_loop; // continue with dispatch loop
|
|
|
|
}
|
|
|
|
|
2014-01-18 18:24:36 -05:00
|
|
|
// set file and line number that the exception occurred at
|
2014-01-30 06:49:18 -05:00
|
|
|
// TODO: don't set traceback for exceptions re-raised by END_FINALLY.
|
|
|
|
// But consider how to handle nested exceptions.
|
2014-04-04 06:52:59 -04:00
|
|
|
// TODO need a better way of not adding traceback to constant objects (right now, just GeneratorExit_obj and MemoryError_obj)
|
|
|
|
if (mp_obj_is_exception_instance(nlr.ret_val) && nlr.ret_val != &mp_const_GeneratorExit_obj && nlr.ret_val != &mp_const_MemoryError_obj) {
|
2014-05-31 09:50:46 -04:00
|
|
|
const byte *code_info = code_state->code_info;
|
2014-01-18 18:24:36 -05:00
|
|
|
machine_uint_t code_info_size = code_info[0] | (code_info[1] << 8) | (code_info[2] << 16) | (code_info[3] << 24);
|
2014-01-19 06:48:48 -05:00
|
|
|
qstr source_file = code_info[4] | (code_info[5] << 8) | (code_info[6] << 16) | (code_info[7] << 24);
|
|
|
|
qstr block_name = code_info[8] | (code_info[9] << 8) | (code_info[10] << 16) | (code_info[11] << 24);
|
2014-01-18 18:24:36 -05:00
|
|
|
machine_uint_t source_line = 1;
|
py, vm: Replace save_ip, save_sp with code_state->{ip, sp}.
This may seem a bit of a risky change, in that it may introduce crazy
bugs with respect to volatile variables in the VM loop. But, I think it
should be fine: code_state points to some external memory, so the
compiler should always read/write to that memory when accessing the
ip/sp variables (ie not put them in registers).
Anyway, it passes all tests and improves on all efficiency fronts: about
2-4% faster (64-bit unix), 16 bytes less stack space per call (64-bit
unix) and slightly less executable size (unix and stmhal).
The reason it's more efficient is save_ip and save_sp were volatile
variables, so were anyway stored on the stack (in memory, not regs).
Thus converting them to code_state->{ip, sp} doesn't cost an extra
memory dereference (except maybe to get code_state, but that can be put
in a register and then made more efficient for other uses of it).
2014-06-01 07:32:28 -04:00
|
|
|
machine_uint_t bc = code_state->ip - code_info - code_info_size;
|
2014-01-19 06:48:48 -05:00
|
|
|
//printf("find %lu %d %d\n", bc, code_info[12], code_info[13]);
|
2014-01-25 06:43:20 -05:00
|
|
|
for (const byte* ci = code_info + 12; *ci && bc >= ((*ci) & 31); ci++) {
|
|
|
|
bc -= *ci & 31;
|
|
|
|
source_line += *ci >> 5;
|
2014-01-18 18:24:36 -05:00
|
|
|
}
|
2014-01-19 07:38:49 -05:00
|
|
|
mp_obj_exception_add_traceback(nlr.ret_val, source_file, source_line, block_name);
|
2014-01-18 18:24:36 -05:00
|
|
|
}
|
|
|
|
|
2013-12-29 11:54:59 -05:00
|
|
|
while (currently_in_except_block) {
|
|
|
|
// nested exception
|
|
|
|
|
2014-03-22 07:49:31 -04:00
|
|
|
assert(exc_sp >= exc_stack);
|
2013-12-29 11:54:59 -05:00
|
|
|
|
|
|
|
// TODO make a proper message for nested exception
|
|
|
|
// at the moment we are just raising the very last exception (the one that caused the nested exception)
|
|
|
|
|
|
|
|
// move up to previous exception handler
|
2014-03-29 17:16:27 -04:00
|
|
|
POP_EXC_BLOCK();
|
2013-12-29 11:54:59 -05:00
|
|
|
}
|
|
|
|
|
2014-03-22 07:49:31 -04:00
|
|
|
if (exc_sp >= exc_stack) {
|
2013-12-29 11:54:59 -05:00
|
|
|
// set flag to indicate that we are now handling an exception
|
|
|
|
currently_in_except_block = 1;
|
|
|
|
|
2013-10-15 17:25:17 -04:00
|
|
|
// catch exception and pass to byte code
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->ip = exc_sp->handler;
|
py: Tidy up variables in VM, probably fixes subtle bugs.
Things get tricky when using the nlr code to catch exceptions. Need to
ensure that the variables (stack layout) in the exception handler are
the same as in the bit protected by the exception handler.
Prior to this patch there were a few bugs. 1) The constant
mp_const_MemoryError_obj was being preloaded to a specific location on
the stack at the start of the function. But this location on the stack
was being overwritten in the opcode loop (since it didn't think that
variable would ever be referenced again), and so when an exception
occurred, the variable holding the address of MemoryError was corrupt.
2) The FOR_ITER opcode detection in the exception handler used sp, which
may or may not contain the right value coming out of the main opcode
loop.
With this patch there is a clear separation of variables used in the
opcode loop and in the exception handler (should fix issue (2) above).
Furthermore, nlr_raise is no longer used in the opcode loop. Instead,
it jumps directly into the exception handler. This tells the C compiler
more about the possible code flow, and means that it should have the
same stack layout for the exception handler. This should fix issue (1)
above. Indeed, the generated (ARM) assembler has been checked explicitly,
and with 'goto exception_handler', the problem with &MemoryError is
fixed.
This may now fix problems with rge-sm, and probably many other subtle
bugs yet to show themselves. Incidentally, rge-sm now passes on
pyboard (with a reduced range of integration)!
Main lesson: nlr is tricky. Don't use nlr_push unless you know what you
are doing! Luckily, it's not used in many places. Using nlr_raise/jump
is fine.
2014-04-17 11:50:23 -04:00
|
|
|
mp_obj_t *sp = MP_TAGPTR_PTR(exc_sp->val_sp);
|
2014-03-29 20:54:48 -04:00
|
|
|
// save this exception in the stack so it can be used in a reraise, if needed
|
|
|
|
exc_sp->prev_exc = nlr.ret_val;
|
2013-10-15 18:46:01 -04:00
|
|
|
// push(traceback, exc-val, exc-type)
|
2013-12-21 13:17:45 -05:00
|
|
|
PUSH(mp_const_none);
|
2013-10-15 18:46:01 -04:00
|
|
|
PUSH(nlr.ret_val);
|
2014-03-28 20:52:17 -04:00
|
|
|
PUSH(mp_obj_get_type(nlr.ret_val));
|
2014-05-31 09:50:46 -04:00
|
|
|
code_state->sp = sp;
|
2013-12-29 11:54:59 -05:00
|
|
|
|
2013-10-15 17:25:17 -04:00
|
|
|
} else {
|
2014-02-15 17:55:00 -05:00
|
|
|
// propagate exception to higher level
|
|
|
|
// TODO what to do about ip and sp? they don't really make sense at this point
|
|
|
|
fastn[0] = nlr.ret_val; // must put exception here because sp is invalid
|
|
|
|
return MP_VM_RETURN_EXCEPTION;
|
2013-10-15 17:25:17 -04:00
|
|
|
}
|
2013-10-04 14:53:11 -04:00
|
|
|
}
|
|
|
|
}
|
|
|
|
}
|