Stream: git-wasmtime

Topic: wasmtime / PR #4572 Cranelift: don't `emit` inside loweri...


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

fitzgen opened PR #4572 from side-effect-free-aarch64-lower to main:

The lowering rules should be "pure", "declarative" and side-effect free, using helpers defined in inst.isle to perform/encapsulate actual side effects like emitting instructions.

See https://github.com/bytecodealliance/wasmtime/blob/main/cranelift/docs/isle-integration.md#lowering-rules-are-always-pure-use-ssa for some more details/guidance here.

cc @dheaton-arm, since you were the original author. Not a big deal, just fyi for the future.

<!--

Please ensure that the following steps are all taken care of before submitting
the PR.

Please ensure all communication adheres to the code of conduct.
-->

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

fitzgen requested cfallin for a review on PR #4572.

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

cfallin submitted PR review.

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

fitzgen merged PR #4572.


Last updated: Dec 23 2024 at 12:05 UTC