Stream: git-wasmtime

Topic: wasmtime / issue #4567 Remove two instruction terminator ...

view this post on Zulip Wasmtime GitHub notifications bot (Jul 31 2022 at 19:31):

bjorn3 opened issue #4567:

Feature

Replace sequences like brz+jump and brnz+jump with single instructions.

Benefit

Simplifies code. These sequences are a leftover from the migration from EBB's to BB's quite a while ago.

Implementation

TBD

Alternatives

Keep the current state at the cost of more complex code.

view this post on Zulip Wasmtime GitHub notifications bot (Aug 01 2022 at 17:19):

cfallin commented on issue #4567:

This would be a nice simplification, I think. One potential issue to check for would be whether InstructionData grows as a result of two branch targets rather than one. Otherwise this fits more nicely with the structure in the rest of the compiler (including branches in VCode).

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 15:43):

akirilov-arm labeled issue #4567:

Feature

Replace sequences like brz+jump and brnz+jump with single instructions.

Benefit

Simplifies code. These sequences are a leftover from the migration from EBB's to BB's quite a while ago.

Implementation

TBD

Alternatives

Keep the current state at the cost of more complex code.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 15:43):

akirilov-arm labeled issue #4567:

Feature

Replace sequences like brz+jump and brnz+jump with single instructions.

Benefit

Simplifies code. These sequences are a leftover from the migration from EBB's to BB's quite a while ago.

Implementation

TBD

Alternatives

Keep the current state at the cost of more complex code.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 02 2022 at 15:43):

akirilov-arm labeled issue #4567:

Feature

Replace sequences like brz+jump and brnz+jump with single instructions.

Benefit

Simplifies code. These sequences are a leftover from the migration from EBB's to BB's quite a while ago.

Implementation

TBD

Alternatives

Keep the current state at the cost of more complex code.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 28 2023 at 16:17):

bjorn3 commented on issue #4567:

This has been fixed. I believe in https://github.com/bytecodealliance/wasmtime/pull/5630.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 28 2023 at 16:17):

bjorn3 closed issue #4567:

Feature

Replace sequences like brz+jump and brnz+jump with single instructions.

Benefit

Simplifies code. These sequences are a leftover from the migration from EBB's to BB's quite a while ago.

Implementation

TBD

Alternatives

Keep the current state at the cost of more complex code.

Last updated: Dec 23 2024 at 12:05 UTC