bytecodealliance / bytecodealliance/wasmtime

Cranelift: Implement the lowering ISLE context with `LowerCtx` directly

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

Description

Right now, it is easy to call Rust from ISLE, but calling back into ISLE from Rust (e.g. to take advantage of optimized sign/zero extend functionality that is in ISLE) is difficult because you don't usually have the whole `IsleContext` anymore.

But you usually have a `LowerCtx`.

And conceptually they are the same: context used when lowering.

Would be nice to combine them into one thing and implement the ISLE lowering context trait with `LowerCtx` directly.

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.