Stream: git-wasmtime

Topic: wasmtime / PR #9532 Remove the `wasmtime_environ::MemoryP...


view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 18:37):

alexcrichton opened PR #9532 from alexcrichton:remove-memory-plan to bytecodealliance:main:

This is the equivalent of #9530 for memories. The goal of this commit is to eventually remove the abstraction layer of MemoryPlan and MemoryStyle in favor of directly reading the configuration of Tunables. The prediction is that it will be simpler to work directly with configured values instead of a layer of abstraction between the configuration and the runtime which needs to be evolved independently to capture how to interpret the configuration.

Like with #9530 my plan is to eventually remove the MemoryStyle type itself, but that'll be a larger change, so it's deferred to a future PR.

<!--
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 (Oct 31 2024 at 18:37):

alexcrichton requested elliottt for a review on PR #9532.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 18:37):

alexcrichton requested wasmtime-compiler-reviewers for a review on PR #9532.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 18:37):

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

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 18:41):

alexcrichton updated PR #9532.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 18:55):

alexcrichton updated PR #9532.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 20:29):

abrown submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Oct 31 2024 at 21:01):

alexcrichton merged PR #9532.


Last updated: Dec 23 2024 at 12:05 UTC