diff --git a/common.mk b/common.mk index c6d6e54c72..bef00d191f 100644 --- a/common.mk +++ b/common.mk @@ -219,6 +219,14 @@ MAKE_LINK = $(MINIRUBY) -rfileutils -e "include FileUtils::Verbose" \ -e "noraise {ln(src, dest)} or" \ -e "cp(src, dest)" +# For release builds +YJIT_RUSTC_ARGS = --crate-name=yjit \ + --crate-type=staticlib \ + --edition=2021 \ + -C opt-level=3 \ + -C overflow-checks=on \ + '--out-dir=$(CARGO_TARGET_DIR)/release/' \ + $(top_srcdir)/yjit/src/lib.rs all: $(SHOWFLAGS) main docs diff --git a/configure.ac b/configure.ac index 8cf0f58067..9823198960 100644 --- a/configure.ac +++ b/configure.ac @@ -4370,6 +4370,10 @@ AC_CONFIG_FILES(Makefile:template/Makefile.in, [ AS_IF([test "$gnumake" != yes], [ echo ['$(MKFILES): $(srcdir)/common.mk'] sed ['s/{\$([^(){}]*)[^{}]*}//g'] ${srcdir}/common.mk + AS_IF([test "$YJIT_SUPPORT" = yes], [ + cat ${srcdir}/yjit/not_gmake.mk + echo ['$(MKFILES): ${srcdir}/yjit/not_gmake.mk'] + ]) ], [ echo 'distclean-local::; @$(RM) GNUmakefile uncommon.mk' ]) @@ -4387,7 +4391,7 @@ AC_CONFIG_FILES(Makefile:template/Makefile.in, [ echo 'ruby: $(PROGRAM);' >> $tmpmk test "$tmpmk" = "$tmpgmk" || rm -f "$tmpgmk" ]) && mv -f $tmpmk Makefile], -[EXEEXT='$EXEEXT' MAKE='${MAKE-make}' gnumake='$gnumake' GIT='$GIT']) +[EXEEXT='$EXEEXT' MAKE='${MAKE-make}' gnumake='$gnumake' GIT='$GIT' YJIT_SUPPORT='$YJIT_SUPPORT']) AC_ARG_WITH([ruby-pc], AS_HELP_STRING([--with-ruby-pc=FILENAME], [pc file basename]), diff --git a/yjit/not_gmake.mk b/yjit/not_gmake.mk new file mode 100644 index 0000000000..cbc60c09f1 --- /dev/null +++ b/yjit/not_gmake.mk @@ -0,0 +1,14 @@ +# This file is included into the Makefile when +# we're *not* using GNU make. Stick to basic features. + +# Rebuild every time since we don't want to list Rust source +# file dependencies. +.PHONY: yjit-static-lib +$(YJIT_LIBS): yjit-static-lib + $(empty) + +yjit-static-lib: + $(ECHO) 'building Rust YJIT (release mode)' + $(Q) $(RUSTC) $(YJIT_RUSTC_ARGS) + +miniruby$(EXEEXT): $(YJIT_LIBS) diff --git a/yjit/yjit.mk b/yjit/yjit.mk index fe182b3dcc..fc0061eb29 100644 --- a/yjit/yjit.mk +++ b/yjit/yjit.mk @@ -23,14 +23,7 @@ YJIT_LIB_TOUCH = touch $@ ifeq ($(YJIT_SUPPORT),yes) $(YJIT_LIBS): $(YJIT_SRC_FILES) $(ECHO) 'building Rust YJIT (release mode)' - $(Q) $(RUSTC) \ - --crate-name=yjit \ - --crate-type=staticlib \ - --edition=2021 \ - -C opt-level=3 \ - -C overflow-checks=on \ - '--out-dir=$(CARGO_TARGET_DIR)/release/' \ - $(top_srcdir)/yjit/src/lib.rs + $(Q) $(RUSTC) $(YJIT_RUSTC_ARGS) $(YJIT_LIB_TOUCH) else ifeq ($(YJIT_SUPPORT),no) $(YJIT_LIBS): @@ -48,7 +41,6 @@ else endif # Put this here instead of in common.mk to avoid breaking nmake builds -# TODO: might need to move for BSD Make support miniruby$(EXEEXT): $(YJIT_LIBS) # By using YJIT_BENCH_OPTS instead of RUN_OPTS, you can skip passing the options to `make install`