Stream: git-wasmtime

Topic: wasmtime / PR #8806 Const-propagate some offsets in `VMOf...


view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 17:35):

alexcrichton opened PR #8806 from alexcrichton:more-const-propagate to bytecodealliance:main:

Before this commit all offsets to all fields in VMOffsets were stored as fields within VMOffset itself. All of the fields at the start of VMOffsets, however, are statically known given the pointer size. Notably this means that the use of HostPtr in the runtime still was forcing a dynamic lookup of these static offsets.

This commit refactors this to reflect all static offsets based solely on the pointer size in the PtrSize trait, removing all the fields from VMOffsets. All the dynamically sized fields, however, remain in VMOffsets.

<!--
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 (Jun 14 2024 at 17:35):

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

view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 17:35):

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

view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 17:35):

alexcrichton requested elliottt for a review on PR #8806.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 17:53):

pchickey submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 18:12):

alexcrichton updated PR #8806.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 18:12):

alexcrichton has enabled auto merge for PR #8806.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 14 2024 at 18:35):

alexcrichton merged PR #8806.


Last updated: Nov 22 2024 at 16:03 UTC