diff options
author | Takashi Kokubun <takashikkbn@gmail.com> | 2023-03-09 22:14:43 -0800 |
---|---|---|
committer | Takashi Kokubun <takashikkbn@gmail.com> | 2023-03-09 22:15:16 -0800 |
commit | 4afe9c09a0cd4688198a5f6f147e5896913200df (patch) | |
tree | 70c58deb4fefdc2cd927322000f9c3f0e347d28f /lib | |
parent | 35fd79ac3713478c8114a498d3536c05ad832063 (diff) |
RJIT: Use imemo_type_p instead
which seems safer. It seems like imemo_type can cause SEGV.
Diffstat (limited to 'lib')
-rw-r--r-- | lib/ruby_vm/rjit/compiler.rb | 5 | ||||
-rw-r--r-- | lib/ruby_vm/rjit/exit_compiler.rb | 2 |
2 files changed, 4 insertions, 3 deletions
diff --git a/lib/ruby_vm/rjit/compiler.rb b/lib/ruby_vm/rjit/compiler.rb index 9e0454edaa..697e7f62d6 100644 --- a/lib/ruby_vm/rjit/compiler.rb +++ b/lib/ruby_vm/rjit/compiler.rb @@ -139,7 +139,7 @@ module RubyVM::RJIT def invalidate_block(block) iseq = block.iseq # Avoid touching GCed ISEQs. We assume it won't be re-entered. - return if C.imemo_type(iseq) != C.imemo_iseq + return unless C.imemo_type_p(iseq, C.imemo_iseq) # Remove this block from the version array remove_block(iseq, block) @@ -296,7 +296,8 @@ module RubyVM::RJIT def rjit_blocks(iseq) # Guard against ISEQ GC at random moments - if C.imemo_type(iseq) != C.imemo_iseq + + unless C.imemo_type_p(iseq, C.imemo_iseq) return Hash.new { |h, k| h[k] = {} } end diff --git a/lib/ruby_vm/rjit/exit_compiler.rb b/lib/ruby_vm/rjit/exit_compiler.rb index 3e1d35d239..35ebf2ddc4 100644 --- a/lib/ruby_vm/rjit/exit_compiler.rb +++ b/lib/ruby_vm/rjit/exit_compiler.rb @@ -83,7 +83,7 @@ module RubyVM::RJIT def compile_branch_stub(ctx, asm, branch_stub, target0_p) # Call rb_rjit_branch_stub_hit iseq = branch_stub.iseq - if C.rjit_opts.dump_disasm && C.imemo_type(iseq) == C.imemo_iseq # Guard against ISEQ GC at random moments + if C.rjit_opts.dump_disasm && C.imemo_type_p(iseq, C.imemo_iseq) # Guard against ISEQ GC at random moments asm.comment("branch stub hit: #{iseq.body.location.label}@#{C.rb_iseq_path(iseq)}:#{iseq_lineno(iseq, target0_p ? branch_stub.target0.pc : branch_stub.target1.pc)}") end asm.mov(:rdi, to_value(branch_stub)) |