th -> ec for vm_set_*_stack.

* vm.c: `th` -> `ec` for the following functions:
  * vm_set_top_stack
  * vm_set_eval_stack
  * vm_set_main_stack
  * vm_cref_new_toplevel


git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@60504 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
This commit is contained in:
ko1 2017-10-28 10:23:58 +00:00
parent f8a9d044f4
commit 9c4542dbaa
3 changed files with 38 additions and 37 deletions

51
vm.c
View File

@ -241,12 +241,13 @@ vm_cref_dup(const rb_cref_t *cref)
} }
static rb_cref_t * static rb_cref_t *
vm_cref_new_toplevel(rb_thread_t *th) vm_cref_new_toplevel(rb_execution_context_t *ec)
{ {
rb_cref_t *cref = vm_cref_new(rb_cObject, METHOD_VISI_PRIVATE /* toplevel visibility is private */, FALSE, NULL, FALSE); rb_cref_t *cref = vm_cref_new(rb_cObject, METHOD_VISI_PRIVATE /* toplevel visibility is private */, FALSE, NULL, FALSE);
VALUE top_wrapper = rb_ec_thread_ptr(ec)->top_wrapper;
if (th->top_wrapper) { if (top_wrapper) {
cref = vm_cref_new(th->top_wrapper, METHOD_VISI_PRIVATE, FALSE, cref, FALSE); cref = vm_cref_new(top_wrapper, METHOD_VISI_PRIVATE, FALSE, cref, FALSE);
} }
return cref; return cref;
@ -255,7 +256,7 @@ vm_cref_new_toplevel(rb_thread_t *th)
rb_cref_t * rb_cref_t *
rb_vm_cref_new_toplevel(void) rb_vm_cref_new_toplevel(void)
{ {
return vm_cref_new_toplevel(GET_THREAD()); return vm_cref_new_toplevel(GET_EC());
} }
static void static void
@ -439,33 +440,33 @@ vm_stat(int argc, VALUE *argv, VALUE self)
/* control stack frame */ /* control stack frame */
static void static void
vm_set_top_stack(rb_thread_t *th, const rb_iseq_t *iseq) vm_set_top_stack(rb_execution_context_t *ec, const rb_iseq_t *iseq)
{ {
if (iseq->body->type != ISEQ_TYPE_TOP) { if (iseq->body->type != ISEQ_TYPE_TOP) {
rb_raise(rb_eTypeError, "Not a toplevel InstructionSequence"); rb_raise(rb_eTypeError, "Not a toplevel InstructionSequence");
} }
/* for return */ /* for return */
vm_push_frame(th->ec, iseq, VM_FRAME_MAGIC_TOP | VM_ENV_FLAG_LOCAL | VM_FRAME_FLAG_FINISH, th->top_self, vm_push_frame(ec, iseq, VM_FRAME_MAGIC_TOP | VM_ENV_FLAG_LOCAL | VM_FRAME_FLAG_FINISH, rb_ec_thread_ptr(ec)->top_self,
VM_BLOCK_HANDLER_NONE, VM_BLOCK_HANDLER_NONE,
(VALUE)vm_cref_new_toplevel(th), /* cref or me */ (VALUE)vm_cref_new_toplevel(ec), /* cref or me */
iseq->body->iseq_encoded, th->ec->cfp->sp, iseq->body->iseq_encoded, ec->cfp->sp,
iseq->body->local_table_size, iseq->body->stack_max); iseq->body->local_table_size, iseq->body->stack_max);
} }
static void static void
vm_set_eval_stack(rb_thread_t * th, const rb_iseq_t *iseq, const rb_cref_t *cref, const struct rb_block *base_block) vm_set_eval_stack(rb_execution_context_t *ec, const rb_iseq_t *iseq, const rb_cref_t *cref, const struct rb_block *base_block)
{ {
vm_push_frame(th->ec, iseq, VM_FRAME_MAGIC_EVAL | VM_FRAME_FLAG_FINISH, vm_push_frame(ec, iseq, VM_FRAME_MAGIC_EVAL | VM_FRAME_FLAG_FINISH,
vm_block_self(base_block), VM_GUARDED_PREV_EP(vm_block_ep(base_block)), vm_block_self(base_block), VM_GUARDED_PREV_EP(vm_block_ep(base_block)),
(VALUE)cref, /* cref or me */ (VALUE)cref, /* cref or me */
iseq->body->iseq_encoded, iseq->body->iseq_encoded,
th->ec->cfp->sp, iseq->body->local_table_size, ec->cfp->sp, iseq->body->local_table_size,
iseq->body->stack_max); iseq->body->stack_max);
} }
static void static void
vm_set_main_stack(rb_thread_t *th, const rb_iseq_t *iseq) vm_set_main_stack(rb_execution_context_t *ec, const rb_iseq_t *iseq)
{ {
VALUE toplevel_binding = rb_const_get(rb_cObject, rb_intern("TOPLEVEL_BINDING")); VALUE toplevel_binding = rb_const_get(rb_cObject, rb_intern("TOPLEVEL_BINDING"));
rb_binding_t *bind; rb_binding_t *bind;
@ -473,11 +474,11 @@ vm_set_main_stack(rb_thread_t *th, const rb_iseq_t *iseq)
GetBindingPtr(toplevel_binding, bind); GetBindingPtr(toplevel_binding, bind);
RUBY_ASSERT_MESG(bind, "TOPLEVEL_BINDING is not built"); RUBY_ASSERT_MESG(bind, "TOPLEVEL_BINDING is not built");
vm_set_eval_stack(th, iseq, 0, &bind->block); vm_set_eval_stack(ec, iseq, 0, &bind->block);
/* save binding */ /* save binding */
if (iseq->body->local_table_size > 0) { if (iseq->body->local_table_size > 0) {
vm_bind_update_env(toplevel_binding, bind, vm_make_env_object(th->ec, th->ec->cfp)); vm_bind_update_env(toplevel_binding, bind, vm_make_env_object(ec, ec->cfp));
} }
} }
@ -930,7 +931,7 @@ rb_binding_add_dynavars(VALUE bindval, rb_binding_t *bind, int dyncount, const I
VALUE realpath = pathobj_realpath(pathobj); VALUE realpath = pathobj_realpath(pathobj);
const struct rb_block *base_block; const struct rb_block *base_block;
const rb_env_t *env; const rb_env_t *env;
rb_thread_t *th = GET_THREAD(); rb_execution_context_t *ec = GET_EC();
const rb_iseq_t *base_iseq, *iseq; const rb_iseq_t *base_iseq, *iseq;
NODE *node = 0, tmp_node; NODE *node = 0, tmp_node;
ID minibuf[4], *dyns = minibuf; ID minibuf[4], *dyns = minibuf;
@ -958,9 +959,9 @@ rb_binding_add_dynavars(VALUE bindval, rb_binding_t *bind, int dyncount, const I
node->u1.tbl = 0; /* reset table */ node->u1.tbl = 0; /* reset table */
ALLOCV_END(idtmp); ALLOCV_END(idtmp);
vm_set_eval_stack(th, iseq, 0, base_block); vm_set_eval_stack(ec, iseq, 0, base_block);
vm_bind_update_env(bindval, bind, envval = vm_make_env_object(th->ec, th->ec->cfp)); vm_bind_update_env(bindval, bind, envval = vm_make_env_object(ec, ec->cfp));
rb_vm_pop_frame(th->ec); rb_vm_pop_frame(ec);
env = (const rb_env_t *)envval; env = (const rb_env_t *)envval;
return env->env; return env->env;
@ -2030,7 +2031,7 @@ rb_iseq_eval(const rb_iseq_t *iseq)
{ {
rb_thread_t *th = GET_THREAD(); rb_thread_t *th = GET_THREAD();
VALUE val; VALUE val;
vm_set_top_stack(th, iseq); vm_set_top_stack(th->ec, iseq);
val = vm_exec(th); val = vm_exec(th);
return val; return val;
} }
@ -2041,7 +2042,7 @@ rb_iseq_eval_main(const rb_iseq_t *iseq)
rb_thread_t *th = GET_THREAD(); rb_thread_t *th = GET_THREAD();
VALUE val; VALUE val;
vm_set_main_stack(th, iseq); vm_set_main_stack(th->ec, iseq);
val = vm_exec(th); val = vm_exec(th);
return val; return val;
} }
@ -2102,19 +2103,19 @@ VALUE
rb_vm_call_cfunc(VALUE recv, VALUE (*func)(VALUE), VALUE arg, rb_vm_call_cfunc(VALUE recv, VALUE (*func)(VALUE), VALUE arg,
VALUE block_handler, VALUE filename) VALUE block_handler, VALUE filename)
{ {
rb_thread_t *th = GET_THREAD(); rb_execution_context_t *ec = GET_EC();
const rb_control_frame_t *reg_cfp = th->ec->cfp; const rb_control_frame_t *reg_cfp = ec->cfp;
const rb_iseq_t *iseq = rb_iseq_new(0, filename, filename, Qnil, 0, ISEQ_TYPE_TOP); const rb_iseq_t *iseq = rb_iseq_new(0, filename, filename, Qnil, 0, ISEQ_TYPE_TOP);
VALUE val; VALUE val;
vm_push_frame(th->ec, iseq, VM_FRAME_MAGIC_TOP | VM_ENV_FLAG_LOCAL | VM_FRAME_FLAG_FINISH, vm_push_frame(ec, iseq, VM_FRAME_MAGIC_TOP | VM_ENV_FLAG_LOCAL | VM_FRAME_FLAG_FINISH,
recv, block_handler, recv, block_handler,
(VALUE)vm_cref_new_toplevel(th), /* cref or me */ (VALUE)vm_cref_new_toplevel(ec), /* cref or me */
0, reg_cfp->sp, 0, 0); 0, reg_cfp->sp, 0, 0);
val = (*func)(arg); val = (*func)(arg);
rb_vm_pop_frame(th->ec); rb_vm_pop_frame(ec);
return val; return val;
} }

View File

@ -21,7 +21,7 @@ static inline VALUE vm_yield(rb_execution_context_t *ec, int argc, const VALUE *
static inline VALUE vm_yield_with_block(rb_execution_context_t *ec, int argc, const VALUE *argv, VALUE block_handler); static inline VALUE vm_yield_with_block(rb_execution_context_t *ec, int argc, const VALUE *argv, VALUE block_handler);
static inline VALUE vm_yield_force_blockarg(rb_execution_context_t *ec, VALUE args); static inline VALUE vm_yield_force_blockarg(rb_execution_context_t *ec, VALUE args);
static VALUE vm_exec(rb_thread_t *th); static VALUE vm_exec(rb_thread_t *th);
static void vm_set_eval_stack(rb_thread_t * th, const rb_iseq_t *iseq, const rb_cref_t *cref, const struct rb_block *base_block); static void vm_set_eval_stack(rb_execution_context_t * th, const rb_iseq_t *iseq, const rb_cref_t *cref, const struct rb_block *base_block);
static int vm_collect_local_variables_in_heap(const VALUE *dfp, const struct local_var_list *vars); static int vm_collect_local_variables_in_heap(const VALUE *dfp, const struct local_var_list *vars);
static VALUE rb_eUncaughtThrow; static VALUE rb_eUncaughtThrow;
@ -1263,7 +1263,7 @@ eval_string_with_cref(VALUE self, VALUE src, VALUE scope, rb_cref_t *const cref_
{ {
int state; int state;
VALUE result = Qundef; VALUE result = Qundef;
rb_thread_t *volatile th = GET_THREAD(); rb_execution_context_t *ec = GET_EC();
struct rb_block block; struct rb_block block;
const struct rb_block *base_block; const struct rb_block *base_block;
volatile VALUE file; volatile VALUE file;
@ -1294,7 +1294,7 @@ eval_string_with_cref(VALUE self, VALUE src, VALUE scope, rb_cref_t *const cref_
base_block = &bind->block; base_block = &bind->block;
} }
else { else {
rb_control_frame_t *cfp = rb_vm_get_ruby_level_next_cfp(th->ec, th->ec->cfp); rb_control_frame_t *cfp = rb_vm_get_ruby_level_next_cfp(ec, ec->cfp);
if (cfp != 0) { if (cfp != 0) {
block.as.captured = *VM_CFP_TO_CAPTURED_BLOCK(cfp); block.as.captured = *VM_CFP_TO_CAPTURED_BLOCK(cfp);
@ -1316,7 +1316,7 @@ eval_string_with_cref(VALUE self, VALUE src, VALUE scope, rb_cref_t *const cref_
iseq = rb_iseq_compile_with_option(src, fname, realpath, INT2FIX(line), base_block, Qnil); iseq = rb_iseq_compile_with_option(src, fname, realpath, INT2FIX(line), base_block, Qnil);
if (!iseq) { if (!iseq) {
rb_exc_raise(adjust_backtrace_in_eval(th, th->ec->errinfo)); rb_exc_raise(adjust_backtrace_in_eval(rb_ec_thread_ptr(ec), ec->errinfo));
} }
/* TODO: what the code checking? */ /* TODO: what the code checking? */
@ -1329,7 +1329,7 @@ eval_string_with_cref(VALUE self, VALUE src, VALUE scope, rb_cref_t *const cref_
cref = NULL; /* use stacked CREF */ cref = NULL; /* use stacked CREF */
} }
} }
vm_set_eval_stack(th, iseq, cref, base_block); vm_set_eval_stack(ec, iseq, cref, base_block);
if (0) { /* for debug */ if (0) { /* for debug */
VALUE disasm = rb_iseq_disasm(iseq); VALUE disasm = rb_iseq_disasm(iseq);
@ -1338,26 +1338,26 @@ eval_string_with_cref(VALUE self, VALUE src, VALUE scope, rb_cref_t *const cref_
/* save new env */ /* save new env */
if (bind && iseq->body->local_table_size > 0) { if (bind && iseq->body->local_table_size > 0) {
vm_bind_update_env(scope, bind, vm_make_env_object(th->ec, th->ec->cfp)); vm_bind_update_env(scope, bind, vm_make_env_object(ec, ec->cfp));
} }
} }
if (file != Qundef) { if (file != Qundef) {
/* kick */ /* kick */
return vm_exec(th); return vm_exec(rb_ec_thread_ptr(ec));
} }
EC_PUSH_TAG(th->ec); EC_PUSH_TAG(ec);
if ((state = EC_EXEC_TAG()) == TAG_NONE) { if ((state = EC_EXEC_TAG()) == TAG_NONE) {
result = vm_exec(th); result = vm_exec(rb_ec_thread_ptr(ec));
} }
EC_POP_TAG(); EC_POP_TAG();
if (state) { if (state) {
if (state == TAG_RAISE) { if (state == TAG_RAISE) {
adjust_backtrace_in_eval(th, th->ec->errinfo); adjust_backtrace_in_eval(rb_ec_thread_ptr(ec), ec->errinfo);
} }
EC_JUMP_TAG(th->ec, state); EC_JUMP_TAG(ec, state);
} }
return result; return result;
} }

View File

@ -243,7 +243,7 @@ method_definition_set(const rb_method_entry_t *me, rb_method_definition_t *def,
method_cref = cref; method_cref = cref;
} }
else { else {
method_cref = vm_cref_new_toplevel(GET_THREAD()); /* TODO: can we reuse? */ method_cref = vm_cref_new_toplevel(GET_EC()); /* TODO: can we reuse? */
} }
RB_OBJ_WRITE(me, &def->body.iseq.cref, method_cref); RB_OBJ_WRITE(me, &def->body.iseq.cref, method_cref);