alexcrichton opened PR #11035 from alexcrichton:x64-cmpxchg to bytecodealliance:main:
A few new operands were needed to define this as well as a number of new instruction "shapes" in a few locations, but nothing too too radical.
<!--
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.htmlPlease ensure all communication follows the code of conduct:
https://github.com/bytecodealliance/wasmtime/blob/main/CODE_OF_CONDUCT.md
-->
alexcrichton requested cfallin for a review on PR #11035.
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #11035.
alexcrichton updated PR #11035.
alexcrichton submitted PR review.
alexcrichton created PR review comment:
TIL there's multiple possible encodings for one instruction:
0: 66 f0 45 0f b1 19 lock cmpxchg %r11w,(%r9) 6: f0 66 45 0f b1 19 lock cmpxchg %r11w,(%r9)
fitzgen submitted PR review.
alexcrichton has enabled auto merge for PR #11035.
alexcrichton merged PR #11035.
abrown submitted PR review.
abrown created PR review comment:
I'm trying to document some of this stuff as I come across it:
Last updated: Dec 06 2025 at 06:05 UTC