-
Notifications
You must be signed in to change notification settings - Fork 107
Unimplemented x86 llvm intrinsic vcvtph2ps #1545
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
Comments
Support for f16 and f128 is basically non-existent in Cranelift so I can't implement these intrinsics the regular way of emulating it using scalar operations. Instead I did have to use inline asm to directly emit the respective x86 instructions, which makes compilation slower as it involves spawning another rustc instance with the LLVM backend to do the compilation of assembly for us. I have some ideas about how to handle this better in the future, but those ideas are not something that will be implemented within a couple of weeks. |
(some more info at #1461) |
@bjorn3 Does Cranelift want to eventually support f16 and f128 more directly or is the asm route the plan for the foreseeable future? |
Cranelift will probably have to support it eventually, but the asm route will be much quicker. |
I have also such warnings at the start
but app crashes in a little different place, so not sure if this is the same issue
|
When compiling half I noticed two warnings for a missing x86 llvm intrinsic vcvtph2ps:
Thanks for all your work on the cranelift backend for rust!
Edit: I guess this is might be related to #1461
The text was updated successfully, but these errors were encountered: