Skip to content

Commit 7ad062a

Browse files
authored
Lock rust toolchain on v1.76.0 because higher versions segfaults during PGO gather (#46)
1 parent 04ba564 commit 7ad062a

2 files changed

Lines changed: 11 additions & 11 deletions

File tree

.github/workflows/codspeed.yml

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -22,9 +22,9 @@ jobs:
2222
- name: Install python dependencies.
2323
run: pip install -r pytests/requirements.txt
2424

25-
- name: Install rust stable.
25+
- name: Install rust.
2626
id: rust-toolchain
27-
uses: dtolnay/rust-toolchain@stable
27+
uses: dtolnay/rust-toolchain@1.76.0
2828
with:
2929
components: llvm-tools
3030

@@ -40,7 +40,7 @@ jobs:
4040
run: pytest . --benchmark-enable
4141

4242
- name: Prepare merged PGO data.
43-
run: rustup run stable bash -c '$RUSTUP_HOME/toolchains/$RUSTUP_TOOLCHAIN/lib/rustlib/x86_64-unknown-linux-gnu/bin/llvm-profdata merge -o ${{ github.workspace }}/merged.profdata ${{ github.workspace }}/profdata'
43+
run: rustup run 1.76.0 bash -c '$RUSTUP_HOME/toolchains/$RUSTUP_TOOLCHAIN/lib/rustlib/x86_64-unknown-linux-gnu/bin/llvm-profdata merge -o ${{ github.workspace }}/merged.profdata ${{ github.workspace }}/profdata'
4444

4545
- name: Compile with profile.
4646
run: pip install -v -e .

.github/workflows/release.yml

Lines changed: 8 additions & 8 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,9 @@ jobs:
7878
python-version: '3.11'
7979
architecture: ${{ matrix.python-architecture || 'x64' }}
8080

81-
- name: Install rust stable
81+
- name: Install rust
8282
id: rust-toolchain
83-
uses: dtolnay/rust-toolchain@stable
83+
uses: dtolnay/rust-toolchain@1.76.0
8484
with:
8585
components: llvm-tools
8686

@@ -91,7 +91,7 @@ jobs:
9191
manylinux: ${{ matrix.manylinux || 'auto' }}
9292
container: ${{ matrix.container }}
9393
args: --release --out dist --interpreter ${{ matrix.maturin-interpreter || matrix.interpreter || '3.7 3.8 3.9 3.10 3.11 3.12 pypy3.7 pypy3.8 pypy3.9 pypy3.10' }} ${{ matrix.extra-build-args }}
94-
rust-toolchain: stable
94+
rust-toolchain: 1.76.0
9595
docker-options: -e CI
9696

9797
- name: List wheels.
@@ -131,9 +131,9 @@ jobs:
131131
with:
132132
python-version: ${{ matrix.interpreter }}
133133

134-
- name: Install rust stable.
134+
- name: Install rust.
135135
id: rust-toolchain
136-
uses: dtolnay/rust-toolchain@stable
136+
uses: dtolnay/rust-toolchain@1.76.0
137137
with:
138138
components: llvm-tools
139139

@@ -146,7 +146,7 @@ jobs:
146146
with:
147147
manylinux: auto
148148
args: --release --out pgo-wheel --interpreter ${{ matrix.interpreter }}
149-
rust-toolchain: stable
149+
rust-toolchain: 1.76.0
150150
docker-options: -e CI
151151
env:
152152
RUSTFLAGS: "-Cprofile-generate=${{ github.workspace }}/profdata"
@@ -160,7 +160,7 @@ jobs:
160160
pytest . --benchmark-enable
161161
162162
# we can't use github.workspace here because of Windows with backslashes
163-
rustup run stable bash -c 'echo LLVM_PROFDATA=$RUSTUP_HOME/toolchains/$RUSTUP_TOOLCHAIN/lib/rustlib/${{ env.RUST_HOST }}/bin/llvm-profdata >> "$GITHUB_ENV"'
163+
rustup run 1.76.0 bash -c 'echo LLVM_PROFDATA=$RUSTUP_HOME/toolchains/$RUSTUP_TOOLCHAIN/lib/rustlib/${{ env.RUST_HOST }}/bin/llvm-profdata >> "$GITHUB_ENV"'
164164
165165
- name: Prepare merged PGO data.
166166
run: ${{ env.LLVM_PROFDATA }} merge -o ${{ github.workspace }}/merged.profdata ${{ github.workspace }}/profdata
@@ -170,7 +170,7 @@ jobs:
170170
with:
171171
manylinux: auto
172172
args: --release --out dist --interpreter ${{ matrix.interpreter }}
173-
rust-toolchain: stable
173+
rust-toolchain: 1.76.0
174174
docker-options: -e CI
175175
env:
176176
RUSTFLAGS: "-Cprofile-use=${{ github.workspace }}/merged.profdata"

0 commit comments

Comments
 (0)