bytecodealliance / bytecodealliance/wasmtime

Cranelift: Simplify all our various CLIF load and store instructions

Open
#6,056 0 comments 4 reactions 0 assignees View on GitHub
cranelift cranelift:area:clif
Dominant language
Rust
Stars
18.6k
Forks
1.8k
Avg merge
1d 18h
Merged PRs (30d)
126

Description

* `{u,s}load{8,16,32}` these are all equivalent to `load.i{8,16,32}` followed by `{u,s}extend`. We should just use that larger sequence and backends that have special lowerings for `uload8()` that are better than `uextend(load8())` should add rules to match the compound pattern.

* Ditto for `{u,s}store{8,16,32}`.

* Same for our SIMD loads like `uload16x4` et al. These can be `load.i16x4` followed by a lane-wise `uextend`, etc.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.