[bjorn3]

LLVM intrinsics must always use extern "llvm-intrinsic" and extern "llvm-intrinsic" must only be used for actual LLVM intrinsics.

Follow up to #161398

[rustbot]

r? @ShoyuVanilla

rustbot has assigned @ShoyuVanilla.
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: compiler
• compiler expanded to 75 candidates
• Random selection from 21 candidates

[bjorn3]

Not sure if this is the best place to do this check.

[ShoyuVanilla]

Sorry, I'm pretty ignorant about this area 😅

r? compiler

[dingxiangfei2009]

I reckon that the patch is incomplete. Are you going to "sanitize" the extern "llvm-intrinsics" declarations so that we don't hit the assertions? I can wait a bit.

[bjorn3]

Both extern "llvm-intrinsic" and #[link_name = "llvm.*"] are unstable features marked as internal. As such I think it is fine to ICE whenever you abuse them per rust-lang/compiler-team#620. But if you really want I can change them to fatal errors.

[rustbot]

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.

[bjorn3]

Gentle ping @dingxiangfei2009