Stream: git-wasmtime

Topic: wasmtime / PR #5101 Remove rayon dependency of cranelift-...

view this post on Zulip Wasmtime GitHub notifications bot (Oct 23 2022 at 12:02):

bjorn3 opened PR #5101 from no_isle_rayon to main:

Using rayon adds a lot of dependencies to Cranelift. The total unparallelized time the code that uses rayon takes is less than half a second and it runs at compile time, so there is pretty much no benefit to parallelizing it.

See https://github.com/bytecodealliance/wasmtime/pull/4906/files#r1002697557. I think I'm going to revert back to Cranelift 0.88.1 until this is merged.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 23 2022 at 12:05):

bjorn3 updated PR #5101 from no_isle_rayon to main.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 23 2022 at 18:45):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 23 2022 at 22:13):

elliottt merged PR #5101.

Last updated: Nov 22 2024 at 17:03 UTC