Stream: git-wasmtime

Topic: wasmtime / PR #7520 cranelift: Fix a PCC validation bug w...


view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 22:06):

elliottt opened PR #7520 from elliottt:trevor/pcc-xmm-load-fix to bytecodealliance:main:

Loads from memory to xmm registers using vmovss and vmovsd don't move a full 128 bits, and instead move either 32 or 64 depending on the instruction. This PR changes the current behavior by special casing these two and adjusting the size of the load, correcting the checks for floating point loads. This doesn't fix the problem generally, as this pr only fixes the Vex case, but it's a step towards stabilization.

Co-Authored-By: Nick Fitzgerald <fitzgen@gmail.com>

<!--
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 (Nov 09 2023 at 22:06):

elliottt requested fitzgen for a review on PR #7520.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 22:06):

elliottt requested wasmtime-compiler-reviewers for a review on PR #7520.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 22:48):

fitzgen submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Nov 09 2023 at 23:26):

fitzgen merged PR #7520.


Last updated: Nov 22 2024 at 16:03 UTC