diff options
| author | Takashi Kokubun <[email protected]> | 2022-10-24 09:20:44 -0700 |
|---|---|---|
| committer | <[email protected]> | 2022-10-24 12:20:44 -0400 |
| commit | 120b747b7df25722c982ff4ba0f8be9a1463bd78 () | |
| tree | 15798f8657eabb2a9a8faa5c8dcbcb94cba00c9a | |
| parent | 841be6392ef85a02ab8ce7f323b990d6bbcc09cb (diff) | |
YJIT: Lazily enable YJIT after prelude (#6597)
* YJIT: Lazily enable YJIT after prelude * Update dependencies * Use a bit field for opt->yjit
Notes: Merged-By: maximecb <[email protected]>
| -rw-r--r-- | common.mk | 1 | ||||
| -rw-r--r-- | internal/cmdlineopt.h | 3 | ||||
| -rw-r--r-- | ruby.c | 33 | ||||
| -rw-r--r-- | version.c | 13 |
4 files changed, 29 insertions, 21 deletions
@@ -16771,6 +16771,7 @@ version.$(OBJEXT): $(hdrdir)/ruby.h version.$(OBJEXT): $(hdrdir)/ruby/ruby.h version.$(OBJEXT): $(hdrdir)/ruby/version.h version.$(OBJEXT): $(top_srcdir)/internal/array.h version.$(OBJEXT): $(top_srcdir)/internal/compilers.h version.$(OBJEXT): $(top_srcdir)/internal/gc.h version.$(OBJEXT): $(top_srcdir)/internal/imemo.h @@ -36,6 +36,9 @@ typedef struct ruby_cmdline_options { unsigned int do_split: 1; unsigned int do_search: 1; unsigned int setids: 2; } ruby_cmdline_options_t; struct ruby_opt_message { @@ -73,7 +73,7 @@ STATIC_ASSERT(Qundef_1bit_from_Qnil, singlebit_only_p(Qundef^Qnil)); # define O_ACCMODE (O_RDONLY | O_WRONLY | O_RDWR) #endif -void Init_ruby_description(void); #ifndef HAVE_STDLIB_H char *getenv(); @@ -1614,13 +1614,8 @@ ruby_opt_init(ruby_cmdline_options_t *opt) #if USE_MJIT // rb_call_builtin_inits depends on RubyVM::MJIT.enabled? - if (opt->mjit.on) { mjit_enabled = true; - // rb_threadptr_root_fiber_setup for the initial thread is called before rb_yjit_enabled_p() - // or mjit_enabled becomes true, meaning jit_cont_new is skipped for the initial root fiber. - // Therefore we need to call this again here to set the initial root fiber's jit_cont. - rb_jit_cont_init(); // must be after mjit_enabled = true - } #endif Init_ext(); /* load statically linked extensions before rubygems */ @@ -1628,11 +1623,20 @@ ruby_opt_init(ruby_cmdline_options_t *opt) rb_call_builtin_inits(); ruby_init_prelude(); #if USE_MJIT - // mjit_init is safe only after rb_call_builtin_inits defines RubyVM::MJIT::Compiler if (opt->mjit.on) mjit_init(&opt->mjit); #endif ruby_set_script_name(opt->script_name); require_libraries(&opt->req_list); @@ -1916,22 +1920,15 @@ process_options(int argc, char **argv, ruby_cmdline_options_t *opt) #if USE_MJIT if (FEATURE_SET_P(opt->features, mjit)) { - opt->mjit.on = true; // set mjit.on for ruby_show_version() API and check to call mjit_init() } #endif #if USE_YJIT if (FEATURE_SET_P(opt->features, yjit)) { - rb_yjit_init(); - // rb_threadptr_root_fiber_setup for the initial thread is called before rb_yjit_enabled_p() - // or mjit_enabled becomes true, meaning jit_cont_new is skipped for the initial root fiber. - // Therefore we need to call this again here to set the initial root fiber's jit_cont. - rb_jit_cont_init(); // must be after rb_yjit_init(), but before parsing options raises an exception. } #endif -#if USE_MJIT - mjit_opts.on = opt->mjit.on; /* used by Init_ruby_description(). mjit_init() still can't be called here. */ -#endif - Init_ruby_description(); if (opt->dump & (DUMP_BIT(version) | DUMP_BIT(version_v))) { ruby_show_version(); if (opt->dump & DUMP_BIT(version)) return Qtrue; @@ -9,6 +9,7 @@ **********************************************************************/ #include "ruby/ruby.h" #include "version.h" #include "vm_core.h" @@ -121,13 +122,19 @@ Init_version(void) } #if USE_MJIT -#define MJIT_OPTS_ON mjit_opts.on #else #define MJIT_OPTS_ON 0 #endif void -Init_ruby_description(void) { VALUE description; @@ -135,7 +142,7 @@ Init_ruby_description(void) rb_dynamic_description = ruby_description_with_mjit; description = MKSTR(description_with_mjit); } - else if (rb_yjit_enabled_p()) { rb_dynamic_description = ruby_description_with_yjit; description = MKSTR(description_with_yjit); } |