In cranelift, a duplicate of the rustc FxHash is used in some places
1: as the rustc implementation is now on crates.io (rustc-hash) would it make sense to use that (or would it not be possible due to vetting)
2: otherwise would it be possible to separate out the current hash fn from the codegen crate so it could also be used elsewhere (eg in Isle compilation)
I've used the fxhash
crate before, which seems to be the same thing. Unclear to me the differences here, if any. All else being equal, I'd prefer to depend on the version that is owned by the rust-lang team.
vetting, as you mention, would be required but that doesn't mean this is impossible, just that a core maintainer would have to find the time to do the vet
in general, I'd prefer option (1) over (2)
Yh definitely understand preferring 1 over 2: could open a PR turning the current vendored version into the rustc one?
yeah, I can commit to doing an audit of the crate if you make that PR
PR is now open (tried pinging on GitHub but not sure if that worked) https://github.com/bytecodealliance/wasmtime/pull/8498
Last updated: Dec 23 2024 at 13:07 UTC