Skip to content

Dep graph cleanups#156175

Merged
rust-bors[bot] merged 3 commits intorust-lang:mainfrom
nnethercote:dep_graph-cleanups
May 8, 2026
Merged

Dep graph cleanups#156175
rust-bors[bot] merged 3 commits intorust-lang:mainfrom
nnethercote:dep_graph-cleanups

Conversation

@nnethercote
Copy link
Copy Markdown
Contributor

Minor cleanups I found while looking at dep graph code. Details in individual commits.

r? @adwinwhite

The code is simpler with it gone.
@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 May 5, 2026
Comment thread compiler/rustc_middle/src/dep_graph/graph.rs Outdated
By removing "random number", which is no longer true. And also wrapping
the overly long lines.
To make clear some things I wasn't sure about.
@nnethercote nnethercote force-pushed the dep_graph-cleanups branch from 50a8566 to 57e0d9d Compare May 5, 2026 22:56
@adwinwhite
Copy link
Copy Markdown
Contributor

Thanks!

@bors r+ rollup

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 8, 2026

📌 Commit 57e0d9d has been approved by adwinwhite

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 May 8, 2026
rust-bors Bot pushed a commit that referenced this pull request May 8, 2026
…uwer

Rollup of 7 pull requests

Successful merges:

 - #156236 (resolve: Remove `MacroData`)
 - #156298 (Rename the unstable integer `extend` function to `widen`)
 - #154498 (turn some long-deprecated -C options into errors)
 - #155734 (Reject outer attributes on `cfg_select` branches)
 - #156123 (Simplify the creation of synthetic HIR.)
 - #156175 (Dep graph cleanups)
 - #156214 (Do not cache `lints_that_dont_need_to_run` across sessions)
rust-bors Bot pushed a commit that referenced this pull request May 8, 2026
…uwer

Rollup of 7 pull requests

Successful merges:

 - #156236 (resolve: Remove `MacroData`)
 - #156298 (Rename the unstable integer `extend` function to `widen`)
 - #154498 (turn some long-deprecated -C options into errors)
 - #155734 (Reject outer attributes on `cfg_select` branches)
 - #156123 (Simplify the creation of synthetic HIR.)
 - #156175 (Dep graph cleanups)
 - #156214 (Do not cache `lints_that_dont_need_to_run` across sessions)
@rust-bors rust-bors Bot merged commit 91d4f8b into rust-lang:main May 8, 2026
11 checks passed
@rustbot rustbot added this to the 1.97.0 milestone May 8, 2026
rust-timer added a commit that referenced this pull request May 8, 2026
Rollup merge of #156175 - nnethercote:dep_graph-cleanups, r=adwinwhite

Dep graph cleanups

Minor cleanups I found while looking at dep graph code. Details in individual commits.

r? @adwinwhite
@nnethercote nnethercote deleted the dep_graph-cleanups branch May 10, 2026 22:30
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.

4 participants