diff options
| author | Alan Wu <XrXr@users.noreply.github.com> | 2023-09-07 14:56:07 -0400 |
|---|---|---|
| committer | Alan Wu <XrXr@users.noreply.github.com> | 2023-09-12 11:25:07 -0400 |
| commit | 39ee3e22bd3d071c1c283b6b8dbd1af413342fb1 (patch) | |
| tree | da6a4f4c93d1887d477776b7f4575d5146f702e4 /yjit | |
| parent | b90272b3b661bd2fd557332028b8feb568b6b9df (diff) | |
Make Kernel#lambda raise when given non-literal block
Previously, Kernel#lambda returned a non-lambda proc when given a
non-literal block and issued a warning under the `:deprecated` category.
With this change, Kernel#lambda will always return a lambda proc, if it
returns without raising.
Due to interactions with block passing optimizations, we previously had
two separate code paths for detecting whether Kernel#lambda got a
literal block. This change allows us to remove one path, the hack done
with rb_control_frame_t::block_code introduced in 85a337f for supporting
situations where Kernel#lambda returned a non-lambda proc.
[Feature #19777]
Co-authored-by: Takashi Kokubun <takashikkbn@gmail.com>
Notes
Notes:
Merged: https://github.com/ruby/ruby/pull/8405
Diffstat (limited to 'yjit')
| -rw-r--r-- | yjit/src/codegen.rs | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/yjit/src/codegen.rs b/yjit/src/codegen.rs index 69b1c52c9c..3ae519bc81 100644 --- a/yjit/src/codegen.rs +++ b/yjit/src/codegen.rs @@ -5194,9 +5194,6 @@ fn gen_push_frame( let block_handler = asm.load( Opnd::mem(64, ep_opnd, SIZEOF_VALUE_I32 * VM_ENV_DATA_INDEX_SPECVAL) ); - - asm.store(Opnd::mem(64, CFP, RUBY_OFFSET_CFP_BLOCK_CODE), block_handler); - block_handler } BlockHandler::AlreadySet => 0.into(), // unused |