Skip to content

Revert "Revert "Use LLVM intrinsics for madd intrinsics""#2018

Merged
folkertdev merged 2 commits intomainfrom
revert-2014-llvm-22-madd
Feb 5, 2026
Merged

Revert "Revert "Use LLVM intrinsics for madd intrinsics""#2018
folkertdev merged 2 commits intomainfrom
revert-2014-llvm-22-madd

Conversation

@folkertdev
Copy link
Contributor

Reverts #2014

While LLVM 22 fixes the adler32 pattern of a useless multiply by one, it does not handle a pattern found in base64 where a shift is used instead of a multiply.

@folkertdev folkertdev marked this pull request as ready for review February 2, 2026 17:08
@rustbot
Copy link
Collaborator

rustbot commented Feb 2, 2026

r? @sayantn

rustbot has assigned @sayantn.
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

@folkertdev
Copy link
Contributor Author

linking llvm/llvm-project#179326

@sayantn sayantn added this pull request to the merge queue Feb 5, 2026
@github-merge-queue github-merge-queue bot removed this pull request from the merge queue due to failed status checks Feb 5, 2026
@folkertdev folkertdev added this pull request to the merge queue Feb 5, 2026
Merged via the queue into main with commit 6d61dc3 Feb 5, 2026
75 checks passed
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.

3 participants