Skip to content

Improve the ABI between the panic runtime and libstd - #161048

Open
bjorn3 wants to merge 5 commits into
rust-lang:mainfrom
bjorn3:refactor_unwind5
Open

Improve the ABI between the panic runtime and libstd#161048
bjorn3 wants to merge 5 commits into
rust-lang:mainfrom
bjorn3:refactor_unwind5

Conversation

@bjorn3

@bjorn3 bjorn3 commented Aug 13, 2026

Copy link
Copy Markdown
Member
  • Use the Rust ABI rather than C ABI
  • Mark some extern functions as safe
  • Move PanicPayload from libcore to liballoc and directly reference Box<dyn Any> instead of *mut dyn Any
  • Avoid an unnecessary allocation with panic=abort

Follow up to #160440

@rustbot rustbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue. labels Aug 13, 2026
@rustbot

rustbot commented Aug 13, 2026

Copy link
Copy Markdown
Collaborator

r? @clarfonthey

rustbot has assigned @clarfonthey.
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: libs
  • libs expanded to 12 candidates
  • Random selection from JohnTitor, Mark-Simulacrum, clarfonthey, nia-e

Comment thread library/alloc/src/panicking.rs Outdated
@rustbot

rustbot commented Aug 14, 2026

Copy link
Copy Markdown
Collaborator

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

bjorn3 commented Aug 14, 2026

Copy link
Copy Markdown
Member Author

#160440 has been merged, so this is ready for review.

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

Labels

S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. T-libs Relevant to the library team, which will review and decide on the PR/issue.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants