Stream: git-wasmtime

Topic: wasmtime / PR #3785 [RFC] ISLE: Migrate call and return i...


view this post on Zulip Wasmtime GitHub notifications bot (Feb 09 2022 at 13:29):

uweigand opened PR #3785 from isle-callret to main:

This adds infrastructure to allow implementing call and return
instructions in ISLE, and migrates the s390x back-end.

Not intended to be committed as-is, this will not even compile
as it depends on the following pre-requisite patches:
https://github.com/bytecodealliance/wasmtime/pull/3783
https://github.com/bytecodealliance/wasmtime/pull/3784

Note that the s390x back end never requires multiple slots for
a single argument - the infrastructure to handle this should
already be present, however.

This uses ABICallerIsle instead of the existing ABICaller.
The new type is used solely to collect information about how
to pass arguments and return values - all the actual code
generation is done in ISLE rules. (Note that ABICallerIsle
ended up as just a thin wrapper around ABISig with public
accessors - maybe the two should be merged?)

To implement loops in ISLE rule, this patch uses regular tail
recursion, employing a Range data structure holding a range
of integers to be looped over.

@cfallin @fitzgen - this is my current state of the call/ret patch - FYI and discussion welcome!

<!--

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 (Feb 25 2022 at 00:11):

uweigand updated PR #3785 from isle-callret to main.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 07 2022 at 19:08):

cfallin submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (May 16 2022 at 16:59):

uweigand updated PR #3785 from isle-callret to main.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 28 2022 at 12:06):

uweigand updated PR #3785 from isle-callret to main.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 28 2022 at 17:12):

uweigand updated PR #3785 from isle-callret to main.

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

cfallin submitted PR review.

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

cfallin merged PR #3785.


Last updated: Dec 23 2024 at 12:05 UTC