summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorTakashi Kokubun <takashikkbn@gmail.com>2022-06-15 15:57:27 -0700
committerTakashi Kokubun <takashikkbn@gmail.com>2022-06-15 15:57:30 -0700
commit5310147bb826424059f258b76b37bebdc0abbc63 (patch)
tree71f92c2a85ec890b8454bbac33327ec29da12a32
parent1e8fed2d2a3c42e007c830648a5f6427b99283d0 (diff)
MJIT: Handle JIT failure properly
-rw-r--r--mjit.c10
-rw-r--r--test/ruby/test_mjit.rb12
2 files changed, 21 insertions, 1 deletions
diff --git a/mjit.c b/mjit.c
index c94a2cfc86..3bceccfb4c 100644
--- a/mjit.c
+++ b/mjit.c
@@ -244,7 +244,15 @@ check_unit_queue(void)
current_cc_ms = real_ms_time();
current_cc_unit = unit;
current_cc_pid = start_mjit_compile(unit);
- // TODO: handle -1
+
+ // JIT failure
+ if (current_cc_pid == -1) {
+ current_cc_pid = 0;
+ current_cc_unit->iseq->body->jit_func = (mjit_func_t)NOT_COMPILED_JIT_ISEQ_FUNC;
+ current_cc_unit = NULL;
+ return;
+ }
+
if (mjit_opts.wait) {
mjit_wait(unit->iseq->body);
}
diff --git a/test/ruby/test_mjit.rb b/test/ruby/test_mjit.rb
index 09503456e1..7fb984a024 100644
--- a/test/ruby/test_mjit.rb
+++ b/test/ruby/test_mjit.rb
@@ -1198,6 +1198,18 @@ class TestMJIT < Test::Unit::TestCase
end
end if defined?(fork)
+ def test_jit_failure
+ _, err = eval_with_jit("#{<<~"begin;"}\n#{<<~"end;"}", min_calls: 1, verbose: 1)
+ begin;
+ 1.times do
+ class A
+ end
+ end
+ end;
+ assert_match(/^MJIT warning: .+ unsupported instruction: defineclass/, err)
+ assert_match(/^JIT failure: block in <main>/, err)
+ end
+
private
# The shortest way to test one proc