afonso360 opened PR #8536 from afonso360:riscv-zfa
to bytecodealliance:main
:
:wave: Hey,
This PR adds initial support for the Zfa extension and implements the
fminm
/fmaxm
instructions and lowerings.The Zfa extension provides additional floating point related instructions not included in either F/D extensions. Here's a link to version 1 of the extension if anyone feels up to some light reading.
The
f{min,max}m
instructions implemented in this PR are pretty much the same as the basefmin
/fmax
present in the ISA, but they have different NaN behavior, that conveniently matches the semantics of Cranelift'sfmin
/fmax
instructions, allowing for a more efficient lowering.
afonso360 requested elliottt for a review on PR #8536.
afonso360 requested wasmtime-compiler-reviewers for a review on PR #8536.
afonso360 requested wasmtime-default-reviewers for a review on PR #8536.
afonso360 updated PR #8536.
github-actions[bot] commented on PR #8536:
Subscribe to Label Action
cc @cfallin, @fitzgen
<details>
This issue or pull request has been labeled: "cranelift", "cranelift:area:riscv64", "cranelift:meta", "isle"Thus the following users have been cc'd because of the following labels:
- cfallin: isle
- fitzgen: isle
To subscribe or unsubscribe from this label, edit the <code>.github/subscribe-to-label.json</code> configuration file.
Learn more.
</details>
alexcrichton submitted PR review:
Nice!
alexcrichton merged PR #8536.
Last updated: Nov 22 2024 at 16:03 UTC