-
Notifications
You must be signed in to change notification settings - Fork 107
Unimplemented x86 llvm intrinsic pcmpestrm128 #1418
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
That is quite a complex intrinsic: https://www.intel.com/content/www/us/en/docs/intrinsics-guide/index.html#text=_mm_cmpestri&ig_expand=939 I don't think I will get around implementing it in the next couple of days.
❤️ |
Implemented this using inline asm to directly call the original instruction over at https://github.com/rust-lang/rustc_codegen_cranelift/tree/more_simd_intrinsics |
Thanks a lot, tested it and it works :) |
A fix is available on the latest nightly. |
Great that you managed to merge it! With new instruction set extensions getting published almost every year this topic never is really done, but rather a nice goal post would be to cover all instructions set extensions older than X years. |
So far I have been implementing new vendor intrinsics as people report them actually being used. There are a lot of them that basically nobody uses. Unless I know of a user to test it with, it is hard for me to be confident that I correctly implemented a intrinsic. If you find other unimplemented intrinsics that some program or library needs, please do let me know. |
When compiling
hard-xml
I noticed two warnings for a missing x86 llvm intrinsicpcmpestrm128
:Edit: And of course a huge thank you for your work and congratulations for getting it into rustup!
The text was updated successfully, but these errors were encountered: