diff options
author | Peter Zhu <peter@peterzhu.ca> | 2023-03-21 10:51:35 -0400 |
---|---|---|
committer | Peter Zhu <peter@peterzhu.ca> | 2023-03-21 10:51:35 -0400 |
commit | 30e7561d1d21844bd6fc7a2ced12cd08cf3ea5ea (patch) | |
tree | f18eb73c250a44be082435e7ac84952c1e9fcb30 /yjit/src/codegen.rs | |
parent | f67f0d72688679267c4040c5f6aa8c493067704a (diff) | |
download | ruby-30e7561d1d21844bd6fc7a2ced12cd08cf3ea5ea.tar.gz |
Revert "YJIT: Rest and block_arg support (#7557)"
This reverts commit 5d0a1ffafa61da04dbda38a5cb5565bcb8032a78.
This commit is causing sequel in yjit-bench to raise with this stack trace:
```
sequel-5.64.0/lib/sequel/dataset/sql.rb:266:in `literal': wrong argument type Array (expected Proc) (TypeError)
from sequel-5.64.0/lib/sequel/database/misc.rb:269:in `literal'
from sequel-5.64.0/lib/sequel/adapters/shared/sqlite.rb:314:in `column_definition_default_sql'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:564:in `block in column_definition_sql'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:564:in `each'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:564:in `column_definition_sql'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:634:in `block in column_list_sql'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:634:in `map'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:634:in `column_list_sql'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:753:in `create_table_sql'
from sequel-5.64.0/lib/sequel/adapters/shared/sqlite.rb:348:in `create_table_sql'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:702:in `create_table_from_generator'
from sequel-5.64.0/lib/sequel/database/schema_methods.rb:203:in `create_table'
from benchmarks/sequel/benchmark.rb:19:in `<main>'
```
Diffstat (limited to 'yjit/src/codegen.rs')
-rw-r--r-- | yjit/src/codegen.rs | 9 |
1 files changed, 7 insertions, 2 deletions
diff --git a/yjit/src/codegen.rs b/yjit/src/codegen.rs index 6b840ee475..d0fd439264 100644 --- a/yjit/src/codegen.rs +++ b/yjit/src/codegen.rs @@ -5181,9 +5181,9 @@ fn move_rest_args_to_stack(array: Opnd, num_args: u32, ctx: &mut Context, asm: & let array_len_opnd = get_array_len(asm, array); - asm.comment("Side exit if length is less than required"); + asm.comment("Side exit if length doesn't not equal remaining args"); asm.cmp(array_len_opnd, num_args.into()); - asm.jl(counted_exit!(ocb, side_exit, send_iseq_has_rest_and_splat_not_equal)); + asm.jbe(counted_exit!(ocb, side_exit, send_splatarray_length_not_equal)); asm.comment("Push arguments from array"); @@ -5413,6 +5413,11 @@ fn gen_send_iseq( return CantCompile; } + if iseq_has_rest && unsafe { get_iseq_flags_has_block(iseq) } { + gen_counter_incr!(asm, send_iseq_has_rest_and_block); + return CantCompile; + } + if iseq_has_rest && unsafe { get_iseq_flags_has_kw(iseq) } { gen_counter_incr!(asm, send_iseq_has_rest_and_kw); return CantCompile; |