diff options
author | Koichi Sasada <[email protected]> | 2023-07-31 16:04:16 +0900 |
---|---|---|
committer | Koichi Sasada <[email protected]> | 2023-07-31 17:13:43 +0900 |
commit | 280419d0e0ba3e96e19551c70cba789fbedd80e1 () | |
tree | b77175e523f51fb120bcd30a7c2624407bf7381e /vm_method.c | |
parent | e40f8bbd20eb65d14992f1f7dce12b4c5edf614e (diff) |
`calling->cd` instead of `calling->ci`
`struct rb_calling_info::cd` is introduced and `rb_calling_info::ci` is replaced with it to manipulate the inline cache of iseq while method invocation process. So that `ci` can be acessed with `calling->cd->ci`. It adds one indirection but it can be justified by the following points: 1) `vm_search_method_fastpath()` doesn't need `ci` and also `vm_call_iseq_setup_normal()` doesn't need `ci`. It means reducing `cd->ci` access in `vm_sendish()` can make it faster. 2) most of method types need to access `ci` once in theory so that 1 additional indirection doesn't matter.
Notes: Merged: https://.com/ruby/ruby/pull/8129
-rw-r--r-- | vm_method.c | 1 |
1 files changed, 1 insertions, 0 deletions
@@ -322,6 +322,7 @@ invalidate_all_cc(void *vstart, void *vend, size_t stride, void *data) RCLASS_CC_TBL(v) = NULL; } } if (ptr) { asan_poison_object(v); } |