Stream: git-wasmtime

Topic: wasmtime / PR #3784 [RFC] ISLE: Re-implement ValueSlice


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

uweigand opened PR #3784 from isle-valuelist to main:

The current definition of ValueSlice is not usable, since any call to
a constructor returning a ValueSlice will extend the mutable borrow
on the context taken by the constructor call, with the result that it
cannot be passed to any other constructor ever.

Re-implement ValueSlice as a pair of a ValueList identifer plus an
offset into the list. This type can simply be copied without requiring
a borrow on the context.

CC @cfallin @fitzgen

<!--

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 24 2022 at 22:09):

uweigand updated PR #3784 from isle-valuelist to main.

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

cfallin submitted PR review.

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

cfallin submitted PR review.

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

cfallin created PR review comment:

s/is/its/

view this post on Zulip Wasmtime GitHub notifications bot (Feb 24 2022 at 22:31):

uweigand updated PR #3784 from isle-valuelist to main.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 24 2022 at 22:32):

uweigand submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 24 2022 at 22:32):

uweigand created PR review comment:

Fixed, thanks!

view this post on Zulip Wasmtime GitHub notifications bot (Feb 24 2022 at 22:43):

uweigand updated PR #3784 from isle-valuelist to main.

view this post on Zulip Wasmtime GitHub notifications bot (Feb 24 2022 at 23:24):

cfallin merged PR #3784.


Last updated: Dec 23 2024 at 12:05 UTC