diff options
| author | shyouhei <shyouhei@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-09-12 01:55:00 +0000 |
|---|---|---|
| committer | shyouhei <shyouhei@b2dd03c8-39d4-4d8f-98ff-823fe69b080e> | 2018-09-12 01:55:00 +0000 |
| commit | 90fcac16acc37a0edfbf8b038aafd1d1eaa4a988 (patch) | |
| tree | 6632eedc091bc87ee8c58a8a972426a7dff48ee2 /vm_insnhelper.c | |
| parent | 09142b565562b44c1ebaf27cc7716541deec9bec (diff) | |
| download | ruby-90fcac16acc37a0edfbf8b038aafd1d1eaa4a988.tar.gz | |
make opt_case_dispatch leaf
This instruction can be written without rb_funcall. It not only boosts
performance of case statements, but also makes room of future JIT
improvements. Because opt_case_dispatch is about optimization this
should not be a bad thing to have.
----
trunk: ruby 2.6.0dev (2018-09-05 trunk 64634) [x86_64-darwin15]
ours: ruby 2.6.0dev (2018-09-12 leaf-insn 64688) [x86_64-darwin15]
last_commit=make opt_case_dispatch leaf
Calculating -------------------------------------
trunk ours
vm2_case_lit 1.366 2.012 i/s - 1.000 times in 0.731839s 0.497008s
Comparison:
vm2_case_lit
ours: 2.0 i/s
trunk: 1.4 i/s - 1.47x slower
git-svn-id: svn+ssh://ci.ruby-lang.org/ruby/trunk@64689 b2dd03c8-39d4-4d8f-98ff-823fe69b080e
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 8d4b087874..50d95610c3 100644 --- a/vm_insnhelper.c +++ b/vm_insnhelper.c @@ -3355,7 +3355,7 @@ vm_case_dispatch(CDHASH hash, OFFSET else_offset, VALUE key) } } if (st_lookup(RHASH_TBL_RAW(hash), key, &val)) { - return FIX2INT((VALUE)val); + return FIX2LONG((VALUE)val); } else { return else_offset; |