Revert "Use SIMD intrinsics for vector shifts" - #2217
Conversation
|
r? @adamgemmell rustbot has assigned @adamgemmell. Use Why was this reviewer chosen?The reviewer was selected based on:
|
|
(CI seems to be generally broken, that's not from my PR.) |
|
CI should be fixed if you give this a rebase |
05cca4e to
768db9f
Compare
|
This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed. Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers. |
CI seems broken in new ways now. |
|
I don't know what's going on with that Windows job. The 691 failures seem to all be similar: Those are the same? Also the PR doesn't touch that intrinsic...? |
|
Wtf, it worked on a retry... |
https://github.com/rust-lang/stdarch/actions/runs/34347799442/job/102462592457?pr=2217 |
Spooky. Possibly it's due to fp16 just being broken on windows gnu, see llvm_util.rs and https://gcc.gnu.org/bugzilla/show_bug.cgi?id=115054. On a dev profile where inlining doesn't happen maybe the equality check can fail while the debug print works fine. Perhaps we should skip those tests on windows gnu. |
Reverts #1955 to fix rust-lang/rust#159831 and to fix rust-lang/rust#159801.
Cc @sayantn
No tests because the per-intrinsic codegen is actually fine, it's the combined codegen with other operations where things fall apart.