Stream: git-wasmtime

Topic: wasmtime / PR #7093 cranelift(x64): Swap operands to save...


view this post on Zulip Wasmtime GitHub notifications bot (Sep 26 2023 at 19:08):

fitzgen opened PR #7093 from fitzgen:commutative-avx-instruction-encoding to bytecodealliance:main:

For XmmRmiRVex-format instructions, when the opcode is commutative, the first operand is one of xmm{0..7}, and the second operand is one of xmm{8..15}, then we can swap the operands to save a byte on instruction encoding.

Note for the reviewer: it would be good to double check the opcodes that I've marked as commutative. I went through each of them, but getting a second pair of eyes on this would be very appreciated.

<!--
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 (Sep 26 2023 at 19:08):

fitzgen requested abrown for a review on PR #7093.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 26 2023 at 19:08):

fitzgen requested wasmtime-compiler-reviewers for a review on PR #7093.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 26 2023 at 19:42):

alexcrichton submitted PR review.

view this post on Zulip Wasmtime GitHub notifications bot (Sep 26 2023 at 20:29):

alexcrichton merged PR #7093.


Last updated: Nov 22 2024 at 16:03 UTC