-
Notifications
You must be signed in to change notification settings - Fork 1.4k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add initial f16
and f128
support to the aarch64 backend
#9076
Conversation
6473711
to
a2cbb52
Compare
Subscribe to Label Action
This issue or pull request has been labeled: "cranelift", "cranelift:area:aarch64", "cranelift:meta", "fuzzing", "isle", "wasmtime:api", "wasmtime:config"
Thus the following users have been cc'd because of the following labels:
To subscribe or unsubscribe from this label, edit the |
Label Messager: wasmtime:configIt looks like you are changing Wasmtime's configuration options. Make sure to
To modify this label's message, edit the To add new label messages or remove existing label messages, edit the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sorry for the delay on this one -- all looks reasonable to me. Thanks for this work!
@beetrees this needs a rebase to latest |
a2cbb52
to
b7ea393
Compare
Rebased |
b7ea393
to
b21c5fb
Compare
I've fixed the Clippy warnings. |
This PR adds initial support for passing
f16
andf128
values around to the aarch64 backend. Support is added for theload
,store
,bitcast
,select
,f16const
andf128const
CLIF instructions, as well as thefp16
target feature.As a side effect of this PR generalising aarch64
bitcast
support, this PR fixes (for aarch64) #6104.f16
/f128
issue: #8312