-
Notifications
You must be signed in to change notification settings - Fork 13.3k
x86 MSVC does not use SSE(2) #27646
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
Specifically, in the target spec, we have: https://github.com/rust-lang/rust/blob/master/src/librustc_back/target/i686_pc_windows_msvc.rs#L15, unlike every other |
cc @alexcrichton, who wrote that code. Is this an intentional quirk of the MSVC toolchain? |
Oh dear no this was not intentional at all, I just assumed everything was |
This is in line with other targets. Closes #27646
This is in line with other targets. Closes rust-lang#27646
@cmr noticed this on IRC. On
i686-pc-windows-gnu
and in fact on all otheri686-*
triples (except iOS, which is technically calledi386-*
), a SSE2-capable microarchitecture is assumed. @dotdash said that this was copied from Clang. Not doing so on MSVC might be an accident, and a rather costly one.The text was updated successfully, but these errors were encountered: