diff options
| author | Koichi Sasada <ko1@atdot.net> | 2020-07-03 16:56:44 +0900 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2020-07-03 16:56:44 +0900 |
| commit | a0f12a0258e4020bd657ee80b7d8f22bd33ea223 (patch) | |
| tree | f6ebafa85cba2bddddc090f8d3c3b74c2c9f374b /vm_insnhelper.c | |
| parent | 8655c2e69041cc812d30c2e951a8ac9ea7a60c47 (diff) | |
Use ID instead of GENTRY for gvars. (#3278)
Use ID instead of GENTRY for gvars.
Global variables are compiled into GENTRY (a pointer to struct
rb_global_entry). This patch replace this GENTRY to ID and
make the code simple.
We need to search GENTRY from ID every time (st_lookup), so
additional overhead will be introduced.
However, the performance of accessing global variables is not
important now a day and this simplicity helps Ractor development.
Notes
Notes:
Merged-By: ko1 <ko1@atdot.net>
Diffstat (limited to 'vm_insnhelper.c')
| -rw-r--r-- | vm_insnhelper.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/vm_insnhelper.c b/vm_insnhelper.c index cacddc2b0e..f2cab2f4b5 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -3661,7 +3661,7 @@ vm_defined(rb_execution_context_t *ec, rb_control_frame_t *reg_cfp, rb_num_t op_ klass = vm_get_cbase(GET_EP()); break; case DEFINED_GVAR: - if (rb_gvar_defined(rb_global_entry(SYM2ID(obj)))) { + if (rb_gvar_defined(SYM2ID(obj))) { expr_type = DEFINED_GVAR; } break; |