Stream: git-wasmtime

Topic: wasmtime / PR #6043 Make sequence numbers local to instru...


view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 10:59):

bjorn3 opened PR #6043 from less_seq_nums to main:

This allows renumbering to be localized to a single block where previously it could affect the entire function. Also saves 32bit of overhead per block.

I originally set out to remove sequence numbers entirely to save 32bit of overhead for every instruction, but the dominator tree code uses sequence numbers to find the relative order of instructions within a block.

Builds on top of https://github.com/bytecodealliance/wasmtime/pull/6042. I split both of them to simplify bisection and help with reviewing.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 11:04):

bjorn3 updated PR #6043 from less_seq_nums to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 18:52):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 19:51):

bjorn3 updated PR #6043 from less_seq_nums to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 17 2023 at 21:39):

cfallin merged PR #6043.


Last updated: Dec 23 2024 at 13:07 UTC