1berry/ruby
1
0
Fork 0
Code Issues Packages Projects Releases Wiki Activity

RJIT: Remove an unneeded branch

Browse Source
This commit is contained in:
Takashi Kokubun 2023-04-02 12:28:49 -07:00
parent 3fe134759c
commit 284a0ff96c
1 changed files with 6 additions and 9 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

9
lib/ruby_vm/rjit/insn_compiler.rb
View File

@ -1359,14 +1359,14 @@ module RubyVM::RJIT
cd = C.rb_call_data.new(jit.operand(0)) cd = C.rb_call_data.new(jit.operand(0))
blockiseq = jit.operand(1) blockiseq = jit.operand(1)
block_handler = jit_caller_setup_arg_block(jit, ctx, asm, cd.ci, blockiseq, false) block_handler = jit_caller_setup_arg_block(jit, ctx, asm, cd.ci, blockiseq)
if block_handler == CantCompile if block_handler == CantCompile
return CantCompile return CantCompile
end end
# calling->ci # calling->ci
mid = C.vm_ci_mid(cd.ci) mid = C.vm_ci_mid(cd.ci)
calling = build_calling(ci: cd.ci, block_handler: blockiseq) calling = build_calling(ci: cd.ci, block_handler:)
# vm_sendish # vm_sendish
cme, comptime_recv_klass = jit_search_method(jit, ctx, asm, mid, calling) cme, comptime_recv_klass = jit_search_method(jit, ctx, asm, mid, calling)
@ -3918,7 +3918,7 @@ module RubyVM::RJIT
# @param jit [RubyVM::RJIT::JITState] # @param jit [RubyVM::RJIT::JITState]
# @param ctx [RubyVM::RJIT::Context] # @param ctx [RubyVM::RJIT::Context]
# @param asm [RubyVM::RJIT::Assembler] # @param asm [RubyVM::RJIT::Assembler]
def jit_caller_setup_arg_block(jit, ctx, asm, ci, blockiseq, is_super) def jit_caller_setup_arg_block(jit, ctx, asm, ci, blockiseq)
side_exit = side_exit(jit, ctx) side_exit = side_exit(jit, ctx)
if C.vm_ci_flag(ci) & C::VM_CALL_ARGS_BLOCKARG != 0 if C.vm_ci_flag(ci) & C::VM_CALL_ARGS_BLOCKARG != 0
# TODO: Skip cmp + jne using Context? # TODO: Skip cmp + jne using Context?
@ -3939,15 +3939,12 @@ module RubyVM::RJIT
end end
elsif blockiseq != 0 elsif blockiseq != 0
return blockiseq return blockiseq
else
if is_super
else else
# Not implemented yet. Is this even necessary? # Not implemented yet. Is this even necessary?
asm.incr_counter(:send_block_setup) asm.incr_counter(:send_block_setup)
return CantCompile return CantCompile
end end
end end
end
# vm_search_method # vm_search_method
# @param jit [RubyVM::RJIT::JITState] # @param jit [RubyVM::RJIT::JITState]