Skip to content

extern "custom": add tests - #160805

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
folkertdev:extern-custom-more-tests
Aug 10, 2026
Merged

extern "custom": add tests#160805
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
folkertdev:extern-custom-more-tests

Conversation

@folkertdev

Copy link
Copy Markdown
Contributor

tracking issue: #140829

r? tgross35

@folkertdev folkertdev added the F-abi_custom `#![feature(abi_custom)]` label Aug 9, 2026
@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. labels Aug 9, 2026
@rustbot

rustbot commented Aug 9, 2026

Copy link
Copy Markdown
Collaborator

tgross35 is currently at their maximum review capacity.
They may take a while to respond.

unsafe extern "custom" fn return_alias_unit() -> UnitAlias {
//~^ ERROR invalid signature for `extern "custom"` function
std::arch::naked_asm!("")
}

@folkertdev folkertdev Aug 9, 2026

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

this is just locking in the status quo, we can update it if we decide to change that.

View changes since the review

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Mind putting that in a comment? So it doesn't look like we're intentionally rejecting it

@tgross35 tgross35 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

One comment request then r=me

View changes since this review

unsafe extern "custom" fn return_alias_unit() -> UnitAlias {
//~^ ERROR invalid signature for `extern "custom"` function
std::arch::naked_asm!("")
}

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Mind putting that in a comment? So it doesn't look like we're intentionally rejecting it

@folkertdev
folkertdev force-pushed the extern-custom-more-tests branch from 98a35f2 to da0920b Compare August 9, 2026 20:34
@tgross35

tgross35 commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors

rust-bors Bot commented Aug 9, 2026

Copy link
Copy Markdown
Contributor

📌 Commit da0920b has been approved by tgross35

It is now in the queue for this repository.

@rust-bors rust-bors Bot added S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Aug 9, 2026
jhpratt added a commit to jhpratt/rust that referenced this pull request Aug 9, 2026
…s, r=tgross35

`extern "custom"`: add tests

tracking issue: rust-lang#140829

r? tgross35
rust-bors Bot pushed a commit that referenced this pull request Aug 10, 2026
Rollup of 7 pull requests

Successful merges:

 - #160533 (dirfd dir operations (3/4))
 - #160675 (bootstrap: Remove `PATH_REMAP` from command-line selector handling )
 - #160807 (Update rustc crate rkyv to 0.8.18)
 - #160560 (Add nightly-only support for Cargo unremap trim-paths files in `rust-gdb`)
 - #160804 (Change .expect message on net/parser to follow precondition style)
 - #160805 (`extern "custom"`: add tests)
 - #160816 (Mark const ptr methods and free functions as inline(always) to match *mut)
rust-bors Bot pushed a commit that referenced this pull request Aug 10, 2026
Rollup of 11 pull requests

Successful merges:

 - #160675 (bootstrap: Remove `PATH_REMAP` from command-line selector handling )
 - #160807 (Update rustc crate rkyv to 0.8.18)
 - #159690 (Clarify `--remap-path-scope` impact on `rustc` metadata)
 - #160560 (Add nightly-only support for Cargo unremap trim-paths files in `rust-gdb`)
 - #160608 (normalization rework: clean up projection_ty_core)
 - #160785 (Get rid of LLM disclosure checkboxes)
 - #160804 (Change .expect message on net/parser to follow precondition style)
 - #160805 (`extern "custom"`: add tests)
 - #160816 (Mark const ptr methods and free functions as inline(always) to match *mut)
 - #160820 (Stabilize fs_set_times)
 - #160826 (Rename parse_delimited_token_tree in cfg_select)
@rust-bors
rust-bors Bot merged commit b764577 into rust-lang:main Aug 10, 2026
13 checks passed
@rustbot rustbot added this to the 1.99.0 milestone Aug 10, 2026
rust-timer added a commit that referenced this pull request Aug 10, 2026
Rollup merge of #160805 - folkertdev:extern-custom-more-tests, r=tgross35

`extern "custom"`: add tests

tracking issue: #140829

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

Labels

F-abi_custom `#![feature(abi_custom)]` S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. T-compiler Relevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants