alexcrichton opened PR #11095 from alexcrichton:x64-cmov to bytecodealliance:main:
This commit adds all
cmov*variants from the Intel manual to the new assembler. This then additionally removes theCmovepseudo-inst in favor of these new instructions. One difference from before is that the naming in theCCenum does not exactly match what mnemonics Capstone uses to disassemble. For exampleCC.NBin ISLE corresponds to the Intel instructionCMOVNB. This instruction, however, has the same encoding asCMOVAEand Capstone disassembles asCMOVAE. This means that the instruction selection in ISLE isn't a 1:1 match with mnemonics.This additionally adds support in the assembler ISLE generation to understand that instructions which read EFLAGS generate a
ConsumesFlagsvariant in their instruction helpers.<!--
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 fitzgen for a review on PR #11095.
alexcrichton requested wasmtime-compiler-reviewers for a review on PR #11095.
abrown submitted PR review.
github-actions[bot] commented on PR #11095:
Subscribe to Label Action
cc @saulecabrera
<details>
This issue or pull request has been labeled: "cranelift", "cranelift:area:x64", "cranelift:meta", "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>
alexcrichton merged PR #11095.
Last updated: Dec 06 2025 at 06:05 UTC