Skip to content

Comments

Use clobber_abis corresponding to the called functions in [asm.abi-clobbers.many]'s example.#2170

Open
zachs18 wants to merge 1 commit intorust-lang:masterfrom
zachs18:patch-1
Open

Use clobber_abis corresponding to the called functions in [asm.abi-clobbers.many]'s example.#2170
zachs18 wants to merge 1 commit intorust-lang:masterfrom
zachs18:patch-1

Conversation

@zachs18
Copy link

@zachs18 zachs18 commented Feb 12, 2026

The example calls an extern "sysv64" fn and an extern "win64" fn, but only declares clobber_abi("C"), despite being in the section that says clobber_abi can be specified multiple times.

Use `clobber_abi`s corresponding to the called functions in `[asm.abi-clobbers.many]`'s example
@rustbot rustbot added the S-waiting-on-review Status: The marked PR is awaiting review from a maintainer label Feb 12, 2026
@ehuss
Copy link
Contributor

ehuss commented Feb 16, 2026

Makes sense to me. @Amanieu does this look good to you?

Copy link
Member

@Amanieu Amanieu left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

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

Labels

S-waiting-on-review Status: The marked PR is awaiting review from a maintainer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants