summaryrefslogtreecommitdiff
path: root/vm_insnhelper.c
diff options
context:
space:
mode:
authoryugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2012-02-05 03:27:37 +0000
committeryugui <yugui@b2dd03c8-39d4-4d8f-98ff-823fe69b080e>2012-02-05 03:27:37 +0000
commit83b5388b258fc9c7bb4d9885b447be233f54bd93 (patch)
tree5c98fc11a6dae3dd7ebbedd1e5289c053d1de5c5 /vm_insnhelper.c
parenteb40744739ceeb6229411e43219c7720a0316a57 (diff)
merges r32347 from trunk into ruby_1_9_2.
-- * vm_insnhelper.c (vm_getivar): check vm state version to invalidate inline chache (ivar index). fixes Bug #4926. * vm_insnhelper.c (vm_setivar): ditto. git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/branches/ruby_1_9_2@34429 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
Diffstat (limited to 'vm_insnhelper.c')
-rw-r--r--vm_insnhelper.c8
1 files changed, 6 insertions, 2 deletions
diff --git a/vm_insnhelper.c b/vm_insnhelper.c
index d760d17e96..59e979a57a 100644
--- a/vm_insnhelper.c
+++ b/vm_insnhelper.c
@@ -1255,7 +1255,8 @@ vm_getivar(VALUE obj, ID id, IC ic)
VALUE val = Qundef;
VALUE klass = RBASIC(obj)->klass;
- if (ic->ic_class == klass) {
+ if (LIKELY(ic->ic_class == klass &&
+ ic->ic_vmstat == GET_VM_STATE_VERSION())) {
long index = ic->ic_value.index;
long len = ROBJECT_NUMIV(obj);
VALUE *ptr = ROBJECT_IVPTR(obj);
@@ -1277,6 +1278,7 @@ vm_getivar(VALUE obj, ID id, IC ic)
}
ic->ic_class = klass;
ic->ic_value.index = index;
+ ic->ic_vmstat = GET_VM_STATE_VERSION();
}
}
}
@@ -1309,7 +1311,8 @@ vm_setivar(VALUE obj, ID id, VALUE val, IC ic)
VALUE klass = RBASIC(obj)->klass;
st_data_t index;
- if (ic->ic_class == klass) {
+ if (LIKELY(ic->ic_class == klass &&
+ ic->ic_vmstat == GET_VM_STATE_VERSION())) {
long index = ic->ic_value.index;
long len = ROBJECT_NUMIV(obj);
VALUE *ptr = ROBJECT_IVPTR(obj);
@@ -1325,6 +1328,7 @@ vm_setivar(VALUE obj, ID id, VALUE val, IC ic)
if (iv_index_tbl && st_lookup(iv_index_tbl, (st_data_t)id, &index)) {
ic->ic_class = klass;
ic->ic_value.index = index;
+ ic->ic_vmstat = GET_VM_STATE_VERSION();
}
/* fall through */
}