Skip to content

Conversation

alexcrichton
Copy link
Contributor

The default Rust toolchain use to build Rust projects on OSS-Fuzz, nightly-2024-07-12, is too old to build Wasmtime at this time. This toolchain is held back to match the LLVM version that Clang is using (18) and is one of the last Rust nightly builds that still uses LLVM 18 (nightly-2024-07-30 is the last Rust nightly using 18, nightly-2024-08-01 uses LLVM 19).

In lieu of updating the toolchain for all projects I've opted to only update Wasmtime at this time. I believe this means that Wasmtime will lose coverage information for fuzzing because LLVM tooling is at version 18 whereas Wasmtime will be producing coverage files from LLVM 19. For Wasmtime this is ok as we rely on the fuzzers themselves much more than the retrospective coverage information.

The default Rust toolchain use to build Rust projects on OSS-Fuzz,
`nightly-2024-07-12`, is too old to build Wasmtime at this time.
This toolchain is held back to match the LLVM version that Clang is
using (18) and is one of the last Rust nightly builds that still uses
LLVM 18 (`nightly-2024-07-30` is the last Rust nightly using 18,
`nightly-2024-08-01` uses LLVM 19).

In lieu of updating the toolchain for all projects I've opted to only
update Wasmtime at this time. I believe this means that Wasmtime will
lose coverage information for fuzzing because LLVM tooling is at version
18 whereas Wasmtime will be producing coverage files from LLVM 19. For
Wasmtime this is ok as we rely on the fuzzers themselves much more than
the retrospective coverage information.
Copy link

alexcrichton is either the primary contact or is in the CCs list of projects/wasmtime.
alexcrichton has previously contributed to projects/wasmtime. The previous PR was #12668

@oliverchang oliverchang merged commit 5a93222 into google:master Jan 13, 2025
16 checks passed
@alexcrichton alexcrichton deleted the update-wasmtime-rustc branch January 17, 2025 15:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants