Skip to content

Checked/wrapping/strict/unbounded funnel shifts - #161119

Open
pthariensflame wants to merge 1 commit into
rust-lang:mainfrom
pthariensflame:wrapping-funnel-shifts
Open

Checked/wrapping/strict/unbounded funnel shifts#161119
pthariensflame wants to merge 1 commit into
rust-lang:mainfrom
pthariensflame:wrapping-funnel-shifts

Conversation

@pthariensflame

@pthariensflame pthariensflame commented Aug 15, 2026

Copy link
Copy Markdown
Contributor

This implements the wrapping funnel shifts requested at rust-lang/libs-team#642 (comment) and also some additional forms of funnel shift proposed at rust-lang/libs-team#855; this might interact with #161015, which is attempting to stabilize the original funnel shift methods.

The work in this PR was done entirely by some people inside a human brain, without the use of any LLMs…or much use of an IDE since ours kept crashing during this process and we had to resort to a plain text editor. 😅

This implements the tracking issue #161798.

@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 15, 2026
@rustbot

rustbot commented Aug 15, 2026

Copy link
Copy Markdown
Collaborator

r? @JohnTitor

rustbot has assigned @JohnTitor.
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, tgross35

@tgross35 tgross35 added the S-waiting-on-ACP Status: PR has an ACP and is waiting for the ACP to complete. label Aug 15, 2026
@tgross35

Copy link
Copy Markdown
Member

ACP at rust-lang/libs-team#855

Comment thread library/core/src/num/uint_macros.rs Outdated
@rust-bors

This comment has been minimized.

@pthariensflame pthariensflame changed the title Checked/wrapping/unbounded funnel shifts Checked/wrapping/strict/unbounded funnel shifts Aug 26, 2026
@pthariensflame
pthariensflame force-pushed the wrapping-funnel-shifts branch from 3382d39 to bdf9037 Compare August 26, 2026 03:57
@rustbot

rustbot commented Aug 26, 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.

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

Labels

S-waiting-on-ACP Status: PR has an ACP and is waiting for the ACP to complete. 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