Stream: git-wasmtime

Topic: wasmtime / PR #6255 Refactor BlockLoweringOrder::new


view this post on Zulip Wasmtime GitHub notifications bot (Apr 20 2023 at 20:26):

remlse opened PR #6255 from remlse:separate-lb-to-bindex-gen to bytecodealliance:main:

As discussed on Zulip.

This should be a refactor but may have some performance implications.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 20 2023 at 20:26):

remlse requested elliottt for a review on PR #6255.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 20 2023 at 20:26):

remlse requested wasmtime-compiler-reviewers for a review on PR #6255.

view this post on Zulip Wasmtime GitHub notifications bot (Apr 20 2023 at 21:13):

elliottt submitted PR review:

I caught up on the discussion on zulip, and think this is a nice refactoring towards using chaos mode to permute the block order!

view this post on Zulip Wasmtime GitHub notifications bot (Apr 20 2023 at 22:24):

jameysharp submitted PR review:

Benchmarks look fine, as I hoped.

So this change is a marginal compile-time performance improvement at all sizes of our benchmark programs. All I was really concerned about is that it's not worse, so this result is great!

view this post on Zulip Wasmtime GitHub notifications bot (Apr 20 2023 at 22:55):

jameysharp merged PR #6255.


Last updated: Dec 23 2024 at 12:05 UTC