diff options
author | Koichi Sasada <ko1@atdot.net> | 2020-10-16 15:20:40 +0900 |
---|---|---|
committer | Koichi Sasada <ko1@atdot.net> | 2020-10-17 08:18:04 +0900 |
commit | f6661f50854e0cdccb03ee516a21ce62adf6c802 (patch) | |
tree | 72668d7b8a8cdbe7f84234bdd99e93a9049a7918 /iseq.c | |
parent | 91ec5f9e39cf54dd7a157addb778293853571f13 (diff) |
sync RClass::ext::iv_index_tbl
iv_index_tbl manages instance variable indexes (ID -> index).
This data structure should be synchronized with other ractors
so introduce some VM locks.
This patch also introduced atomic ivar cache used by
set/getinlinecache instructions. To make updating ivar cache (IVC),
we changed iv_index_tbl data structure to manage (ID -> entry)
and an entry points serial and index. IVC points to this entry so
that cache update becomes atomically.
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/3662
Diffstat (limited to 'iseq.c')
-rw-r--r-- | iseq.c | 15 |
1 files changed, 15 insertions, 0 deletions
@@ -23,6 +23,7 @@ #include "id_table.h" #include "internal.h" #include "internal/bits.h" +#include "internal/class.h" #include "internal/compile.h" #include "internal/error.h" #include "internal/file.h" @@ -180,6 +181,20 @@ iseq_extract_values(VALUE *code, size_t pos, iseq_value_itr_t * func, void *data } } break; + case TS_IVC: + { + IVC ivc = (IVC)code[pos + op_no + 1]; + if (ivc->entry) { + if (RB_TYPE_P(ivc->entry->class_value, T_NONE)) { + rb_bug("!! %u", ivc->entry->index); + } + VALUE nv = func(data, ivc->entry->class_value); + if (ivc->entry->class_value != nv) { + ivc->entry->class_value = nv; + } + } + } + break; case TS_ISE: { union iseq_inline_storage_entry *const is = (union iseq_inline_storage_entry *)code[pos + op_no + 1]; |