Stream: git-wasmtime

Topic: wasmtime / PR #4238 Validate alignment in the canonical ABI


view this post on Zulip Wasmtime GitHub notifications bot (Jun 07 2022 at 15:59):

alexcrichton opened PR #4238 from validate-align to main:

This commit updates the lifting and lowering done by Wasmtime to
validate that alignment is all correct. Previously alignment was ignored
because I wasn't sure how this would all work out.

To be extra safe I haven't actually modified any loads/stores and
they're all still unaligned. If this becomes a performance issue we can
investigate aligned loads and stores but otherwise I believe the
requisite locations have been guarded with traps and I've also added
debug asserts to catch possible future mistakes.

<!--

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 (Jun 07 2022 at 17:47):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 07 2022 at 17:50):

alexcrichton updated PR #4238 from validate-align to main.

view this post on Zulip Wasmtime GitHub notifications bot (Jun 07 2022 at 18:34):

alexcrichton merged PR #4238.


Last updated: Dec 23 2024 at 13:07 UTC