Skip to content

Correct llvm intrinsic type parameters in AArch64 - #2262

Open
sayantn wants to merge 1 commit into
rust-lang:mainfrom
sayantn:correct-overloaded-intrinsics
Open

sayantn wants to merge 1 commit into
rust-lang:mainfrom
sayantn:correct-overloaded-intrinsics

Conversation

@sayantn

@sayantn sayantn commented Nov 1, 2026 •

Copy link
Copy Markdown
Contributor

Found a few mistakes, due to a modified version of rust-lang/rust#157145. Expectedly all are in AArch64, since only that module uses overloaded intrinsics heavily. I checked the other modules manually, couldn't find anything, but don't trust me on this. I checked AArch64 and X86 modules using the modified rustc, with --release and --tests so that the assert_instr tests are generated, and since almost all intrinsics have it, this should cover almost everything

@rustbot

rustbot commented Nov 1, 2026

Copy link
Copy Markdown
Collaborator

r? @folkertdev

rustbot has assigned @folkertdev.
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
  • @Amanieu, @adamgemmell, @davidtwco, @folkertdev expanded to Amanieu, adamgemmell, davidtwco, folkertdev
  • Random selection from Amanieu, adamgemmell, davidtwco, folkertdev

This branch has not been deployed

No deployments
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