Stream: git-wasmtime

Topic: wasmtime / PR #8118 x64: Fix load sinking in `rounds{s,d}...


view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2024 at 17:21):

alexcrichton opened PR #8118 from alexcrichton:x64-fix-round-loads to bytecodealliance:main:

This commit aims to address #8116 by fixing these two instructions to load the proper amount of bytes when a load is sunk into them. Currently the instruction variant used here requires an aligned XmmMem which is today auto-translated with a 16-byte load unconditionally. For loads near the end of memory this loads too much and can erroneously cause a trap. The fix in this commit is to force the load to happen manually with the appropriate type rather than a 16-byte type.

I'll note that XmmMem is left as an argument in this case because the AVX variants of these instructions can continue to leverage unaligned accesses. This also means that the test added here won't fail on a machine with AVX support, it needs to be explicitly disabled.

Closes #8116

<!--
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 13 2024 at 17:21):

alexcrichton requested abrown for a review on PR #8118.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2024 at 17:21):

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

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2024 at 17:21):

alexcrichton requested fitzgen for a review on PR #8118.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2024 at 17:21):

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

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2024 at 20:34):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Mar 13 2024 at 21:00):

fitzgen merged PR #8118.


Last updated: Dec 23 2024 at 12:05 UTC