Skip to content

Conversation

@Unique-Usman
Copy link
Contributor

Now that Rc::new_zeroed is stable, remove the cfg(feature = "nightly") branch and the temporary zeroed array on stable. This avoids copying a potentially large [Word; CHUNK_WORDS] into the Rc.

Now that Rc::new_zeroed is stable, remove the cfg(feature = "nightly")
branch and the temporary zeroed array on stable. This avoids copying a
potentially large [Word; CHUNK_WORDS] into the Rc.

Signed-off-by: Usman Akinyemi <usmanakinyemi202@gmail.com>
@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 Jan 15, 2026
@rustbot
Copy link
Collaborator

rustbot commented Jan 15, 2026

r? @jdonszelmann

rustbot has assigned @jdonszelmann.
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

@Unique-Usman
Copy link
Contributor Author

Rc::new_zeroed got stabilized through this #149619

Copy link
Member

@Kivooeo Kivooeo left a comment

Choose a reason for hiding this comment

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

no idea why it was here, but yeah looks fine to me

View changes since this review

@lqd
Copy link
Member

lqd commented Jan 15, 2026

no idea why it was here

Because r-a uses this crate via rustc_type_ir and it needs to build on stable.

@Kivooeo
Copy link
Member

Kivooeo commented Jan 15, 2026

ah, i see, thanks for explanation!

@Kivooeo
Copy link
Member

Kivooeo commented Jan 16, 2026

r? me

@bors r+

@rust-bors
Copy link
Contributor

rust-bors bot commented Jan 16, 2026

📌 Commit 0dc5b52 has been approved by Kivooeo

It is now in the queue for this repository.

@rustbot rustbot assigned Kivooeo and unassigned jdonszelmann Jan 16, 2026
@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 Jan 16, 2026
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jan 16, 2026
simplify words initialization using Rc::new_zeroed

Now that Rc::new_zeroed is stable, remove the cfg(feature = "nightly") branch and the temporary zeroed array on stable. This avoids copying a potentially large [Word; CHUNK_WORDS] into the Rc.
rust-bors bot pushed a commit that referenced this pull request Jan 16, 2026
Rollup of 6 pull requests

Successful merges:

 - #145354 (Cache derive proc macro expansion with incremental query)
 - #151123 (Support primitives in type info reflection)
 - #151178 (simplify words initialization using Rc::new_zeroed)
 - #151187 (Use `default_field_values` more in `Resolver`)
 - #151197 (remove lcnr from compiler review rotation)
 - #151203 (Revert `QueryStackFrame` split)

r? @ghost
@rust-bors rust-bors bot merged commit 4227404 into rust-lang:main Jan 16, 2026
11 checks passed
@rustbot rustbot added this to the 1.94.0 milestone Jan 16, 2026
rust-timer added a commit that referenced this pull request Jan 16, 2026
Rollup merge of #151178 - ua/fixme, r=Kivooeo

simplify words initialization using Rc::new_zeroed

Now that Rc::new_zeroed is stable, remove the cfg(feature = "nightly") branch and the temporary zeroed array on stable. This avoids copying a potentially large [Word; CHUNK_WORDS] into the Rc.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

5 participants