Stream: git-wasmtime

Topic: wasmtime / PR #12872 Fix a stacked borrows violation in f...


view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 08:08):

alexcrichton requested dicej for a review on PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 08:08):

alexcrichton opened PR #12872 from alexcrichton:fix-future-stream-copy-stacked-borrows to bytecodealliance:main:

This commit fixes an issue in the copy implementation for futures/streams related to component-model-async. Specifically the added tests here tripped an error in Miri which is related to intra-component copies and stacked borrows. This refactoring ends up using copy_within for the intra-component case and copy_to_nonoverlapping for the inter-component case to resolve this issue. This commit additionally refactors the validation/checks to happen in one location instead of in a few to ensure that all validation is done up-front in a shared manner.

<!--
Please make sure you include the following information:

Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 08:08):

alexcrichton requested wasmtime-core-reviewers for a review on PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 08:59):

alexcrichton updated PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 11:03):

github-actions[bot] added the label wasmtime:api on PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 11:34):

alexcrichton requested wasmtime-wasi-reviewers for a review on PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 11:34):

alexcrichton updated PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 14:36):

dicej submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 14:36):

dicej created PR review comment:

Could we retain the comment about why it's safe to use copy_to_nonoverlapping here?

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 15:00):

alexcrichton updated PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 15:01):

alexcrichton has enabled auto merge for PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 15:14):

alexcrichton added PR #12872 Fix a stacked borrows violation in futures/streams copy to the merge queue.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 17:59):

alexcrichton merged PR #12872.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 30 2026 at 17:59):

alexcrichton removed PR #12872 Fix a stacked borrows violation in futures/streams copy from the merge queue.


Last updated: Apr 12 2026 at 23:10 UTC