diff options
author | Alan Wu <XrXr@users.noreply.github.com> | 2021-04-28 12:59:26 -0400 |
---|---|---|
committer | Alan Wu <XrXr@users.noreply.github.com> | 2021-10-20 18:19:34 -0400 |
commit | 36134f7d293b3f02e85599a3374d670aa575aeb0 (patch) | |
tree | 3f2290a0c180a025a373c4e7246be0caf064699f /yjit_iface.h | |
parent | bce6dea72d0081e4777b80e1de3b76fbfcde9f0a (diff) |
Implement calls to methods with simple optional params
* Implement calls to methods with simple optional params
* Remove unnecessary MJIT_STATIC
See comment for MJIT_STATIC. I added it not knowing whether it's
required because the function next to it has it. Don't use it and wait
for problems to come up instead.
* Better naming, some comments
* Count bailing on kw only iseqs
On railsbench:
```
opt_send_without_block exit reasons:
bmethod 59729 (27.7%)
optimized_method 59137 (27.5%)
iseq_complex_callee 41362 (19.2%)
alias_method 33346 (15.5%)
callsite_not_simple 19170 ( 8.9%)
iseq_only_keywords 1300 ( 0.6%)
kw_splat 1299 ( 0.6%)
cfunc_ruby_array_varg 18 ( 0.0%)
```
Diffstat (limited to 'yjit_iface.h')
-rw-r--r-- | yjit_iface.h | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/yjit_iface.h b/yjit_iface.h index eabaff1955..a7a89b7c1f 100644 --- a/yjit_iface.h +++ b/yjit_iface.h @@ -38,8 +38,9 @@ YJIT_DECLARE_COUNTERS( oswb_cfunc_argc_mismatch, oswb_cfunc_toomany_args, oswb_iseq_tailcall, - oswb_iseq_argc_mismatch, - oswb_iseq_not_simple, + oswb_iseq_arity_error, + oswb_iseq_only_keywords, + oswb_iseq_complex_callee, oswb_not_implemented_method, oswb_getter_arity, oswb_se_receiver_not_heap, |