Stream: git-wasmtime

Topic: wasmtime / PR #10082 Winch: Add replace_lane instructions...

Wasmtime GitHub notifications bot (Jan 22 2025 at 21:46):

jeffcharles opened PR #10082 from jeffcharles:winch-simd-replace-lane 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
-->
Part of #8093. Implements the *.replace_lane set of instructions.

Wasmtime GitHub notifications bot (Jan 22 2025 at 21:46):

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

Wasmtime GitHub notifications bot (Jan 22 2025 at 21:46):

jeffcharles requested cfallin for a review on PR #10082.

Wasmtime GitHub notifications bot (Jan 22 2025 at 21:46):

jeffcharles requested fitzgen for a review on PR #10082.

Wasmtime GitHub notifications bot (Jan 22 2025 at 21:46):

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

Wasmtime GitHub notifications bot (Jan 22 2025 at 23:45):

github-actions[bot] commented on PR #10082:

Subscribe to Label Action

cc @saulecabrera

<details>
This issue or pull request has been labeled: "winch"

Thus the following users have been cc'd because of the following labels:

• saulecabrera: winch

To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.

Learn more.
</details>

Wasmtime GitHub notifications bot (Jan 23 2025 at 12:01):

saulecabrera commented on PR #10082:

I can take this review.

Wasmtime GitHub notifications bot (Jan 23 2025 at 12:01):

saulecabrera requested saulecabrera for a review on PR #10082.

Wasmtime GitHub notifications bot (Jan 23 2025 at 15:00):

jeffcharles updated PR #10082.

Wasmtime GitHub notifications bot (Jan 23 2025 at 19:29):

saulecabrera submitted PR review:

LGTM, one minor remark.

Wasmtime GitHub notifications bot (Jan 23 2025 at 19:29):

saulecabrera created PR review comment:

I'd like to keep OperandSize as machine independent as possible. Would it be possible to move this to a helper in the x64 masm/asm?

Last updated: Jan 24 2025 at 00:11 UTC