Skip to content

Revert "x86: use simd::intrinsics for saturating packs" - #2211

Merged
folkertdev merged 1 commit into
mainfrom
revert-2033-saturating_packs
Aug 19, 2026
Merged

Revert "x86: use simd::intrinsics for saturating packs"#2211
folkertdev merged 1 commit into
mainfrom
revert-2033-saturating_packs

Conversation

@folkertdev

Copy link
Copy Markdown
Contributor

Reverts #2033

@rustbot

rustbot commented Aug 19, 2026

Copy link
Copy Markdown
Collaborator

r? @adamgemmell

rustbot has assigned @adamgemmell.
They will have a look at your PR within the next two weeks and either review your PR or reassign to another reviewer.

Use r? to explicitly pick a reviewer

Why was this reviewer chosen?

The reviewer was selected based on:

  • Owners of files modified in this PR: @Amanieu, @adamgemmell, @davidtwco, @folkertdev, @sayantn
  • @Amanieu, @adamgemmell, @davidtwco, @folkertdev, @sayantn expanded to Amanieu, adamgemmell, davidtwco, folkertdev, sayantn
  • Random selection from Amanieu, adamgemmell, davidtwco, sayantn

@folkertdev
folkertdev enabled auto-merge August 19, 2026 17:39
@folkertdev
folkertdev added this pull request to the merge queue Aug 19, 2026
@tgross35

Copy link
Copy Markdown
Member

Any context here?

Merged via the queue into main with commit a1bae01 Aug 19, 2026
82 checks passed
@folkertdev

Copy link
Copy Markdown
Contributor Author

Oh, yes #2208. The pattern doesn't actually optimize well, and we don't have a good way in general to make it work using portable intrinsics right now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants