Stream: git-wasmtime

Topic: wasmtime / PR #5366 Allocate temporary intermediates when...


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

elliottt opened PR #5366 from trevor/ssa-aarch64-load-constant to main:

As loading constants on aarch64 can take up to 4 instructions, we need to plumb through some additional registers. Rather than pass a fixed list of registers in, pass an allocation function.

This forced a change to ABIMachineSpec::gen_memcpy, as it now needed to take an allocator to pass to uses of gen_constant. As a result, I removed the fixed temporary arguments and passed an allocator instead.
<!--

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 (Dec 01 2022 at 19:19):

elliottt submitted PR review.

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

elliottt created PR review comment:

The two consecutive calls to load_constant are what's forcing the alloc_tmp argument to load_constant to be a mutable reference.

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

elliottt updated PR #5366 from trevor/ssa-aarch64-load-constant to main.

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

elliottt requested jameysharp for a review on PR #5366.

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

elliottt requested fitzgen for a review on PR #5366.

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

elliottt created PR review comment:

I need to update the implementation in lower/isle.rs before merging this pr.

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

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 01 2022 at 20:05):

elliottt updated PR #5366 from trevor/ssa-aarch64-load-constant to main.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 01 2022 at 20:16):

elliottt updated PR #5366 from trevor/ssa-aarch64-load-constant to main.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 01 2022 at 20:46):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 01 2022 at 21:55):

elliottt updated PR #5366 from trevor/ssa-aarch64-load-constant to main.

view this post on Zulip Wasmtime GitHub notifications bot (Dec 01 2022 at 21:56):

elliottt has enabled auto merge for PR #5366.

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

elliottt merged PR #5366.


Last updated: Dec 23 2024 at 13:07 UTC