RJIT: Rename pause/resume to disable/enable

like YJIT. They don't work in the same way yet, but it's nice to make
the naming consistent first so that we will not need to rename them
later.
This commit is contained in:
Takashi Kokubun 2023-12-21 14:24:10 -08:00
Родитель 69bed94cea
Коммит c73959cff4
4 изменённых файлов: 7 добавлений и 8 удалений

7
rjit.c
Просмотреть файл

@ -139,9 +139,8 @@ rb_rjit_setup_options(const char *s, struct rb_rjit_options *rjit_opt)
else if (opt_match_noarg(s, l, "verify-ctx")) {
rjit_opt->verify_ctx = true;
}
// --rjit=pause is an undocumented feature for experiments
else if (opt_match_noarg(s, l, "pause")) {
rjit_opt->pause = true;
else if (opt_match_noarg(s, l, "disable")) {
rjit_opt->disable = true;
}
else {
rb_raise(rb_eRuntimeError,
@ -437,7 +436,7 @@ rb_rjit_init(const struct rb_rjit_options *opts)
}
// Enable RJIT and stats from here
rb_rjit_call_p = !rb_rjit_opts.pause;
rb_rjit_call_p = !rb_rjit_opts.disable;
rjit_stats_p = rb_rjit_opts.stats;
}

2
rjit.h
Просмотреть файл

@ -35,7 +35,7 @@ struct rb_rjit_options {
// Verify context objects
bool verify_ctx;
// [experimental] Do not start RJIT until RJIT.resume is called.
bool pause;
bool disable;
};
RUBY_SYMBOL_EXPORT_BEGIN

Просмотреть файл

@ -4,8 +4,8 @@ module RubyVM::RJIT
Primitive.cexpr! 'RBOOL(rb_rjit_enabled)'
end
# Start generating JITed code again after --rjit-pause.
def self.resume
# Start generating JITed code again after --rjit-disable.
def self.enable
Primitive.cstmt! %{
rb_rjit_call_p = true;
return Qnil;

Просмотреть файл

@ -1332,7 +1332,7 @@ module RubyVM::RJIT # :nodoc: all
trace_exits: [self._Bool, Primitive.cexpr!("OFFSETOF((*((struct rb_rjit_options *)NULL)), trace_exits)")],
dump_disasm: [self._Bool, Primitive.cexpr!("OFFSETOF((*((struct rb_rjit_options *)NULL)), dump_disasm)")],
verify_ctx: [self._Bool, Primitive.cexpr!("OFFSETOF((*((struct rb_rjit_options *)NULL)), verify_ctx)")],
pause: [self._Bool, Primitive.cexpr!("OFFSETOF((*((struct rb_rjit_options *)NULL)), pause)")],
disable: [self._Bool, Primitive.cexpr!("OFFSETOF((*((struct rb_rjit_options *)NULL)), disable)")],
)
end