Stream: git-wasmtime

Topic: wasmtime / PR #5679 Cranelift: Introduce the `return_call...


view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 00:33):

fitzgen requested elliottt for a review on PR #5679.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 00:34):

fitzgen opened PR #5679 from tail-call-conv to main:

This is an in-progress implementation of https://github.com/bytecodealliance/rfcs/pull/29. The very non-controversial bits that just introduce a calling convention and tail call instructions, nothing that actually implements the calling conventions or supports lowering. As such, I think it doesn't really need to block on the RFC merging.

We also went down a little bit of a rabbit hole with DFG helper method refactoring. Sorry about the extra bits in the diff!

<!--

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 01 2023 at 00:52):

elliottt created PR review comment:

What do you think about inlining this in the verifier? All the fields used are public on the dfg, and there's currently only one call site.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 00:52):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 00:52):

elliottt submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 00:57):

jameysharp submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 01:06):

jameysharp edited PR review comment.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 01:06):

jameysharp created PR review comment:

Nick and I waffled over that. I kinda felt that it was nice to have it next to DataFlowGraph::inst_result_types since they have nearly identical structures. I don't feel super strongly about it though. I suspect there's some other way of factoring all this that would be even better but I haven't been able to figure out what that would be yet...

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 11:43):

bjorn3 submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 11:43):

bjorn3 created PR review comment:

Can you add this on the other architectures too?

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 20:19):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 20:19):

fitzgen created PR review comment:

The other ISAs don't do exhaustive matches on the calling conventions, so there isn't anywhere to update.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 20:20):

fitzgen updated PR #5679 from tail-call-conv to main.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 20:20):

fitzgen updated PR #5679 from tail-call-conv to main.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 20:21):

fitzgen has enabled auto merge for PR #5679.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 01 2023 at 21:20):

fitzgen merged PR #5679.


Last updated: Dec 23 2024 at 13:07 UTC