From e9e624b76e1f1b2c8dd5d4a6eea7e55ab491822e Mon Sep 17 00:00:00 2001 From: Maxime Chevalier-Boisvert Date: Fri, 2 Dec 2022 15:33:47 -0500 Subject: [PATCH] YJIT: check that we correctly auto-enable YJIT on Linux (#6854) * YJIT: check that we correctly auto-enable YJIT on Linux YJIT should be automatically built on Linux x86-64 when rustc is present, and we should see +YJIT in the version string. * Use miniruby rather than system ruby --- .github/workflows/yjit-ubuntu.yml | 7 ++++++- 1 file changed, 6 insertions(+), 1 deletion(-) diff --git a/.github/workflows/yjit-ubuntu.yml b/.github/workflows/yjit-ubuntu.yml index 6b18872c93..2895afd027 100644 --- a/.github/workflows/yjit-ubuntu.yml +++ b/.github/workflows/yjit-ubuntu.yml @@ -51,7 +51,9 @@ jobs: configure: "--with-gcc=clang-12 --enable-yjit=dev" - test_task: "check" - configure: "--enable-yjit RUSTC='rustc +1.58.0'" # release build + # YJIT should be automatically built in release mode on x86-64 Linux with rustc present + #configure: "--enable-yjit RUSTC='rustc +1.58.0'" + configure: "RUSTC='rustc +1.58.0'" rust_version: "1.58.0" - test_task: "check" @@ -124,6 +126,9 @@ jobs: if: ${{ matrix.test_task == 'check' }} - name: Enable YJIT through ENV run: echo "RUBY_YJIT_ENABLE=1" >> $GITHUB_ENV + # Check that the binary was built with YJIT + - name: Check YJIT enabled + run: ./miniruby --yjit -v | grep "+YJIT" - name: make ${{ matrix.test_task }} run: make -s -j ${{ matrix.test_task }} RUN_OPTS="$RUN_OPTS" YJIT_BENCH_OPTS="$YJIT_BENCH_OPTS" timeout-minutes: 60