Skip to content

Rollup of 2 pull requests#155857

Closed
jhpratt wants to merge 5 commits intorust-lang:mainfrom
jhpratt:rollup-zxYjFnj
Closed

Rollup of 2 pull requests#155857
jhpratt wants to merge 5 commits intorust-lang:mainfrom
jhpratt:rollup-zxYjFnj

Conversation

@jhpratt
Copy link
Copy Markdown
Member

@jhpratt jhpratt commented Apr 27, 2026

Successful merges:

r? @ghost

Create a similar rollup

bjorn3 and others added 5 commits April 26, 2026 21:07
requires_lto is already false by default.
instead of during ast lowering, where it's not easily possible to obtain all the right information in time
… r=fee1-dead

Check closure's constness validity in the constness query

fixes rust-lang#155584

instead of checking during ast lowering, where it's not easily possible to obtain all the right information in time. While lowering an assoc item we don't know if the parent was a const trait or a const impl. Tracing this information is quite annoying, and complicates a lot of code, which checking here after the fact is trivial.
…verride, r=jieyouxu

Remove unnecessary requires_lto: false for eBPF

requires_lto is already false by default.
@rust-bors rust-bors Bot added the rollup A PR which is a rollup label Apr 27, 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 Apr 27, 2026
@jhpratt
Copy link
Copy Markdown
Member Author

jhpratt commented Apr 27, 2026

@bors r+ rollup=never p=5

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Apr 27, 2026

📌 Commit 5a49375 has been approved by jhpratt

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 Apr 27, 2026
@rust-bors

This comment has been minimized.

rust-bors Bot pushed a commit that referenced this pull request Apr 27, 2026
Rollup of 2 pull requests

Successful merges:

 - #155772 (Check closure's constness validity in the constness query)
 - #155839 (Remove unnecessary requires_lto: false for eBPF)
@rust-log-analyzer
Copy link
Copy Markdown
Collaborator

The job x86_64-msvc-ext1 failed! Check out the build log: (web) (plain enhanced) (plain)

Click to see the possible cause of the failure (guessed by this bot)

@rust-bors rust-bors Bot added S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. and removed S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. labels Apr 27, 2026
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Apr 27, 2026

💔 Test for 7b79ee0 failed: CI. Failed job:

@jhpratt jhpratt closed this Apr 27, 2026
@rust-bors rust-bors Bot added the S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. label Apr 27, 2026
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Apr 27, 2026
@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented Apr 27, 2026

This pull request was unapproved due to being closed.

@jhpratt jhpratt deleted the rollup-zxYjFnj branch April 27, 2026 07:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

rollup A PR which is a rollup S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. 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