YJIT: Pass options to ruby/spec with -T
(#10599)
We recently found out that `make test-spec` wasn't testing with `--yjit-call-threshold=1` like the other test suites, and that we need to use `-T` to pass options to it.
This commit is contained in:
parent
470eceff8f
commit
0aefd2f784
3
.github/workflows/yjit-macos.yml
vendored
3
.github/workflows/yjit-macos.yml
vendored
@ -60,11 +60,13 @@ jobs:
|
||||
- test_task: 'check'
|
||||
configure: '--enable-yjit=dev'
|
||||
yjit_opts: '--yjit-call-threshold=1 --yjit-verify-ctx --yjit-code-gc'
|
||||
specopts: '-T --yjit-call-threshold=1 -T --yjit-verify-ctx -T --yjit-code-gc'
|
||||
fail-fast: false
|
||||
|
||||
env:
|
||||
GITPULLOPTIONS: --no-tags origin ${{ github.ref }}
|
||||
RUN_OPTS: ${{ matrix.yjit_opts }}
|
||||
SPECOPTS: ${{ matrix.specopts }}
|
||||
|
||||
runs-on: ${{ github.repository == 'ruby/ruby' && 'macos-arm-oss' || 'macos-14' }}
|
||||
|
||||
@ -128,6 +130,7 @@ jobs:
|
||||
run: >-
|
||||
make -s ${{ matrix.test_task }} ${TESTS:+TESTS="$TESTS"}
|
||||
RUN_OPTS="$RUN_OPTS"
|
||||
SPECOPTS="$SPECOPTS"
|
||||
timeout-minutes: 60
|
||||
env:
|
||||
RUBY_TESTOPTS: '-q --tty=no'
|
||||
|
4
.github/workflows/yjit-ubuntu.yml
vendored
4
.github/workflows/yjit-ubuntu.yml
vendored
@ -95,6 +95,7 @@ jobs:
|
||||
- test_task: 'check'
|
||||
configure: '--enable-yjit=dev'
|
||||
yjit_opts: '--yjit-call-threshold=1 --yjit-verify-ctx --yjit-code-gc'
|
||||
specopts: '-T --yjit-call-threshold=1 -T --yjit-verify-ctx -T --yjit-code-gc'
|
||||
|
||||
- test_task: 'test-bundled-gems'
|
||||
configure: '--enable-yjit=dev'
|
||||
@ -108,6 +109,7 @@ jobs:
|
||||
GITPULLOPTIONS: --no-tags origin ${{ github.ref }}
|
||||
RUN_OPTS: ${{ matrix.yjit_opts }}
|
||||
YJIT_BENCH_OPTS: ${{ matrix.yjit_bench_opts }}
|
||||
SPECOPTS: ${{ matrix.specopts }}
|
||||
RUBY_DEBUG: ci
|
||||
BUNDLE_JOBS: 8 # for yjit-bench
|
||||
RUST_BACKTRACE: 1
|
||||
@ -180,7 +182,7 @@ jobs:
|
||||
- name: make ${{ matrix.test_task }}
|
||||
run: >-
|
||||
make -s ${{ matrix.test_task }} ${TESTS:+TESTS="$TESTS"}
|
||||
RUN_OPTS="$RUN_OPTS" MSPECOPT=--debug
|
||||
RUN_OPTS="$RUN_OPTS" MSPECOPT=--debug SPECOPTS="$SPECOPTS"
|
||||
YJIT_BENCH_OPTS="$YJIT_BENCH_OPTS" YJIT_BINDGEN_DIFF_OPTS="$YJIT_BINDGEN_DIFF_OPTS"
|
||||
timeout-minutes: 90
|
||||
env:
|
||||
|
Loading…
x
Reference in New Issue
Block a user