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
andMemoryStyle
in favor of directly reading the configuration ofTunables
. 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:
If this work has been discussed elsewhere, please include a link to that
conversation. If it was discussed in an issue, just mention "issue #...".Explain why this change is needed. If the details are in an issue already,
this can be brief.Our development process is documented in the Wasmtime book:
https://docs.wasmtime.dev/contributing-development-process.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested elliottt for a review on PR #9532.
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #9532.
alexcrichton requested wasmtime-core-reviewers for a review on PR #9532.
alexcrichton updated PR #9532.
alexcrichton updated PR #9532.
abrown submitted PR review.
alexcrichton merged PR #9532.
Last updated: Nov 22 2024 at 17:03 UTC