Skip to content

Remove unused functions in value_analysis.rs#157004

Merged
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
nnethercote:value_analysis-visibility
May 30, 2026
Merged

Remove unused functions in value_analysis.rs#157004
rust-bors[bot] merged 2 commits into
rust-lang:mainfrom
nnethercote:value_analysis-visibility

Conversation

@nnethercote
Copy link
Copy Markdown
Contributor

And also reduce some visibilities. Details in individual commits.

r? @cjgillot

And reduce visibility of functions only used within the crate.
@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 27, 2026
@cjgillot
Copy link
Copy Markdown
Contributor

@bors r+ rollup

@rust-bors
Copy link
Copy Markdown
Contributor

rust-bors Bot commented May 29, 2026

📌 Commit bdf8d76 has been approved by cjgillot

It is now in the queue for this repository.

🌲 The tree is currently closed for pull requests below priority 1. This pull request will be tested once the tree is reopened.

@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 29, 2026
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…ity, r=cjgillot

Remove unused functions in `value_analysis.rs`

And also reduce some visibilities. Details in individual commits.

r? @cjgillot
JonathanBrouwer added a commit to JonathanBrouwer/rust that referenced this pull request May 30, 2026
…ity, r=cjgillot

Remove unused functions in `value_analysis.rs`

And also reduce some visibilities. Details in individual commits.

r? @cjgillot
rust-bors Bot pushed a commit that referenced this pull request May 30, 2026
…uwer

Rollup of 12 pull requests

Successful merges:

 - #154591 (Remove `will_cache_on_disk_for_key_fn`)
 - #156672 (Misc improvements to coroutine transform code)
 - #157027 (HIR ty lowering: Move some things into submodules)
 - #157051 (Allow two object files for a single CGU in CompiledModule)
 - #157100 (Some more per owner things)
 - #153497 (Use `trait_object_dummy_self` more & heavily fix+update related docs)
 - #155638 (Fix tupled closure signature in `AsyncFn` arg mismatch diagnostic)
 - #156826 (style: Clarify nullary call and `()` no-break rule applies past max width)
 - #157004 (Remove unused functions in `value_analysis.rs`)
 - #157032 (Fixed more &x ->&mut x suggestions)
 - #157033 (Note irrefutable while let in loop type errors)
 - #157139 (compiler: `ops::RangeInclusive` → `range::RangeInclusive`)

Failed merges:

 - #156875 (Correct and document semantics of `yield` terminator)
@rust-bors rust-bors Bot merged commit 1111bad into rust-lang:main May 30, 2026
12 checks passed
@rustbot rustbot added this to the 1.98.0 milestone May 30, 2026
rust-timer added a commit that referenced this pull request May 30, 2026
Rollup merge of #157004 - nnethercote:value_analysis-visibility, r=cjgillot

Remove unused functions in `value_analysis.rs`

And also reduce some visibilities. Details in individual commits.

r? @cjgillot
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.

3 participants