Stream: git-wasmtime

Topic: wasmtime / Issue #1147 Too many raw_bitcasts in SIMD code


view this post on Zulip Wasmtime GitHub notifications bot (Oct 21 2020 at 08:16):

julian-seward1 commented on Issue #1147:

It seems to me that the simplest fix is simply to remove all NxM types from CL's type system, where NxM == 128, and replace them with a single V128 type. What benefit does having all these types bring us? They are not useful for typechecking CLIF that is derived from wasm, since wasm allows free intermixing of the types.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 21 2020 at 08:20):

julian-seward1 commented on Issue #1147:

(notes copied from #2303):

Disadvantages of using bitcasts:

view this post on Zulip Wasmtime GitHub notifications bot (Oct 21 2020 at 08:21):

bjorn3 commented on Issue #1147:

Replacing NxM with V128 would require a new variant of many instructions for every lane size. The current design allows using the same iadd instruction for every integer type both scalar and vector. It is also useful for typechecking CLIF derived from rust, as rust doesn't allow mixing vector types without an explicit transmute.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 21 2020 at 08:31):

julian-seward1 commented on Issue #1147:

One longer-term way around that, once the old backend is no longer needed, that would be to abandon the DSL for defining CLIF instructions. And instead define them using a simple Rust enum, in the same way that the new backends define target specific instructions. And for the vector instructions, include a field of type

enum Laneage { I8X16, I16X8, I32X4, I64X2, F versions of the same, etc }

view this post on Zulip Wasmtime GitHub notifications bot (Oct 21 2020 at 08:32):

julian-seward1 edited a comment on Issue #1147:

One longer-term way around that, once the old backend is no longer needed, would be to abandon the DSL for defining CLIF instructions. And instead define them using a simple Rust enum, in the same way that the new backends define target specific instructions. And for the vector instructions, include a field of type

enum Laneage { I8X16, I16X8, I32X4, I64X2, F versions of the same, etc }

view this post on Zulip Wasmtime GitHub notifications bot (Oct 21 2020 at 08:33):

bjorn3 commented on Issue #1147:

Currently many instructions are both scalar and vector instructions at the same time. For example iadd. Adding a Laneage argument would add unnecessary noise when using scalars only.


Last updated: Dec 23 2024 at 13:07 UTC