Skip to content

Stabilize fN::BITS#154065

Open
Apersoma wants to merge 2 commits into
rust-lang:mainfrom
Apersoma:stabilize-fN-bits
Open

Stabilize fN::BITS#154065
Apersoma wants to merge 2 commits into
rust-lang:mainfrom
Apersoma:stabilize-fN-bits

Conversation

@Apersoma

@Apersoma Apersoma commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

View all comments

Tracking issue: #151073
Closes: #151073

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

rustbot commented Mar 19, 2026

Copy link
Copy Markdown
Collaborator

r? @scottmcm

rustbot has assigned @scottmcm.
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: @scottmcm, libs
  • @scottmcm, libs expanded to 8 candidates
  • Random selection from Mark-Simulacrum, scottmcm

@rustbot

rustbot commented Mar 19, 2026

Copy link
Copy Markdown
Collaborator

compiler-builtins is developed in its own repository. If possible, consider making this change to rust-lang/compiler-builtins instead.

cc @tgross35

@rustbot rustbot added the A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) label Mar 19, 2026
@tgross35 tgross35 assigned tgross35 and unassigned scottmcm Mar 19, 2026
@tgross35

Copy link
Copy Markdown
Contributor

This has only been around since January but it seems pretty unobjectionable to have. However, this does conflict with a constant in use in the ecosystem (showed up in the implementation PR) so we need to take a look at crater.

@bors try

@rustbot label +I-libs-api-nominated

@rustbot rustbot added the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Mar 19, 2026
@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Mar 19, 2026
@rust-bors

rust-bors Bot commented Mar 19, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 0abe69e (0abe69e6c16114311cc2700e972492e64d69c28f, parent: 1f7f8ea0721a3b1eb73e6c6d25cccb371434b320)

@tgross35

Copy link
Copy Markdown
Contributor

Doing a run with tests in case somebody has a different value for BITS for some reason.

@craterbot run mode=build-and-test

@craterbot

Copy link
Copy Markdown
Collaborator

👌 Experiment pr-154065 created and queued.
🤖 Automatically detected try build 0abe69e
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 19, 2026
@Amanieu Amanieu removed the I-libs-api-nominated Nominated for discussion during a libs-api team meeting. label Mar 24, 2026
@Amanieu

Amanieu commented Mar 24, 2026

Copy link
Copy Markdown
Member

FCP started in #151073

@tgross35

Copy link
Copy Markdown
Contributor

@Apersoma please squash the commits and give it a reasonable message, the first commit just says "cmplt" (we don't have squash merging on this repo).

@craterbot

Copy link
Copy Markdown
Collaborator

🚧 Experiment pr-154065 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot

Copy link
Copy Markdown
Collaborator

🎉 Experiment pr-154065 is completed!
📊 297 regressed and 292 fixed (857798 total)
📊 132042 spurious results on the retry-regressed-list.txt, consider a retry1 if this is a significant amount.
📰 Open the summary report.

⚠️ If you notice any spurious failure please add them to the denylist!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

Footnotes

  1. re-run the experiment with crates=https://crater-reports.s3.amazonaws.com/pr-154065/retry-regressed-list.txt

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Mar 28, 2026
@tgross35

Copy link
Copy Markdown
Contributor

@craterbot

Copy link
Copy Markdown
Collaborator

👌 Experiment pr-154065-1 created and queued.
🤖 Automatically detected try build 0abe69e
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Mar 28, 2026
@Apersoma

Copy link
Copy Markdown
Contributor Author

Crater isn't too bad, but there are a couple of crates that I think we could resolve reasonably easy:

Crates that depend on common_traits might run into some an issue too, we'll have to rerun the regressed list once a new version is published.

Let's give some time for fixes here.

@rustbot blocked @SnoozeThis wait 2 months

How do you filter for what's actually useful in crater?

@tgross35

Copy link
Copy Markdown
Contributor

Look for "regressed: dependencies" (some crate in the dep tree failed) and "regressed: root results" (the crate itself failed) in the summary. We retry the spurious results.

You can also go to the downloads tab to grab the results and do something like rg '::BITS' which can be a bit more clear.

@Apersoma

Copy link
Copy Markdown
Contributor Author

This probably going to need a second crater run for errors causes by users of common_traits after the new version gets published, right?

@tgross35

Copy link
Copy Markdown
Contributor

This probably going to need a second crater run for errors causes by users of common_traits after the new version gets published, right?

I think so ^^ 🙂

Crates that depend on common_traits might run into some an issue too, we'll have to rerun the regressed list once a new version is published.

@zommiommy

Copy link
Copy Markdown
Contributor

done, sorry!

@tgross35

Copy link
Copy Markdown
Contributor

Not a problem, there’s no way of knowing until this happens :) thank you for the quick action!

@craterbot

Copy link
Copy Markdown
Collaborator

🚧 Experiment pr-154065-1 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot

Copy link
Copy Markdown
Collaborator

🎉 Experiment pr-154065-1 is completed!
📊 114 regressed and 120 fixed (132208 total)
📊 1656 spurious results on the retry-regressed-list.txt, consider a retry1 if this is a significant amount.
📰 Open the summary report.

⚠️ If you notice any spurious failure please add them to the denylist!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

Footnotes

  1. re-run the experiment with crates=https://crater-reports.s3.amazonaws.com/pr-154065-1/retry-regressed-list.txt

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. S-blocked Status: Blocked on something else such as an RFC or other implementation work. labels Mar 30, 2026
jannic added a commit to jannic/rp-hal that referenced this pull request Apr 11, 2026
jannic added a commit to rp-rs/rp-hal that referenced this pull request Apr 11, 2026
@SnoozeThis

Copy link
Copy Markdown

Resolved.

@rustbot assign @tgross35.

@rustbot rustbot assigned tgross35 and unassigned SnoozeThis May 28, 2026
@rustbot

rustbot commented May 28, 2026

Copy link
Copy Markdown
Collaborator

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

@tgross35

Copy link
Copy Markdown
Contributor

@bors try

@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request May 28, 2026
@rust-bors

rust-bors Bot commented May 28, 2026

Copy link
Copy Markdown
Contributor

☀️ Try build successful (CI)
Build commit: 28f9593 (28f95933dc0727396c07b48d0a8cb5211b7fb472, parent: cced03bfd61a304243a34504618ecec86c17063f)

@tgross35

Copy link
Copy Markdown
Contributor

@craterbot run mode=build-and-test

@craterbot

Copy link
Copy Markdown
Collaborator

👌 Experiment pr-154065-2 created and queued.
🤖 Automatically detected try build 28f9593
🔍 You can check out the queue and this experiment's details.

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot craterbot added S-waiting-on-crater Status: Waiting on a crater run to be completed. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels May 29, 2026
@craterbot

Copy link
Copy Markdown
Collaborator

🚧 Experiment pr-154065-2 is now running

ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

@craterbot

Copy link
Copy Markdown
Collaborator

🎉 Experiment pr-154065-2 is completed!
📊 459 regressed and 426 fixed (954864 total)
📊 5782 spurious results on the retry-regressed-list.txt, consider a retry1 if this is a significant amount.
📰 Open the summary report.

⚠️ If you notice any spurious failure please add them to the denylist!
ℹ️ Crater is a tool to run experiments across parts of the Rust ecosystem. Learn more

Footnotes

  1. re-run the experiment with crates=https://crater-reports.s3.amazonaws.com/pr-154065-2/retry-regressed-list.txt

@craterbot craterbot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-crater Status: Waiting on a crater run to be completed. labels Jun 14, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

A-compiler-builtins Area: compiler-builtins (https://github.com/rust-lang/compiler-builtins) 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.

Tracking Issue for fN::BITS

8 participants