Stream: git-wasmtime

Topic: wasmtime / PR #10127 Winch: Fix Wasm vector loads to oper...

Wasmtime GitHub notifications bot (Jan 27 2025 at 19:54):

jeffcharles opened PR #10127 from jeffcharles:winch-simd-fix-vector-memory-load-size to bytecodealliance:main:

<!--
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.html

Please ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
Fixes #10115. Wasm vector loads operate on 64 bits of memory so update LoadKind::derive_operand_size to return that result for vector loads. This will result in the value containing the offset with access size to be set so that bounds are enforced correctly.

Wasmtime GitHub notifications bot (Jan 27 2025 at 19:54):

jeffcharles requested abrown for a review on PR #10127.

Wasmtime GitHub notifications bot (Jan 27 2025 at 19:54):

jeffcharles requested wasmtime-compiler-reviewers for a review on PR #10127.

Wasmtime GitHub notifications bot (Jan 27 2025 at 19:54):

jeffcharles requested fitzgen for a review on PR #10127.

Wasmtime GitHub notifications bot (Jan 27 2025 at 19:54):

jeffcharles requested wasmtime-core-reviewers for a review on PR #10127.

Wasmtime GitHub notifications bot (Jan 27 2025 at 20:05):

saulecabrera submitted PR review:

Thanks!

Wasmtime GitHub notifications bot (Jan 27 2025 at 20:05):

saulecabrera has enabled auto merge for PR #10127.

Wasmtime GitHub notifications bot (Jan 27 2025 at 20:29):

saulecabrera merged PR #10127.

Last updated: Feb 28 2025 at 02:27 UTC