Skip to content

deduced_param_attrs for unit type offers no benefit - #164020

Merged
rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
tmiasko:deduce-unit
Oct 11, 2026
Merged

rust-bors[bot] merged 1 commit into
rust-lang:mainfrom
tmiasko:deduce-unit

Conversation

@tmiasko

@tmiasko tmiasko commented Oct 9, 2026 •

Copy link
Copy Markdown
Contributor

Outputs, where unit type is implicit default, did participate in the deduction. Identify unit type as irrelevant for the deduction as an optimization.

@rustbot

rustbot commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@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 Oct 9, 2026
@rustbot

rustbot commented Oct 9, 2026

Copy link
Copy Markdown
Collaborator

r? @jackh726

rustbot has assigned @jackh726.
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: compiler, mir, mir-opt
  • compiler, mir, mir-opt expanded to 77 candidates
  • Random selection from 20 candidates

@rust-log-analyzer

This comment has been minimized.

@tmiasko tmiasko closed this Oct 9, 2026
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 9, 2026
@tmiasko tmiasko reopened this Oct 9, 2026
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Oct 9, 2026

@jackh726 jackh726 left a comment •

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

A couple minor nits, but r=me

View changes since this review

Comment thread compiler/rustc_mir_transform/src/deduce_param_attrs.rs Outdated
Comment thread compiler/rustc_mir_transform/src/deduce_param_attrs.rs Outdated
@tmiasko tmiasko changed the title Skip units types in deduced_param_attrs deduced_param_attrs for unit type offers no benefit Oct 10, 2026
Outputs, where unit type is implicit default, did participate in the
deduction. Identify unit type as irrelevant for the deduction as an
optimization.
@jackh726

Copy link
Copy Markdown
Member

@bors r+

@rust-bors

rust-bors Bot commented Oct 11, 2026

Copy link
Copy Markdown
Contributor

📌 Commit ae95e71 has been approved by jackh726

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 Oct 11, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 11, 2026
Rollup of 8 pull requests

Successful merges:

 - #163402 (test-float-parse: Share one table of powers of two sized for f64)
 - #163999 ([tiny] Show trait span for associated item with similar name)
 - #164020 (deduced_param_attrs for unit type offers no benefit)
 - #162926 (fix misleading diagnostics and improve diagnostics of return type mismatch)
 - #164075 (set aliases in closure error bodies to rigid with next-solver)
 - #164100 (More `tests/rustdoc-html` cleanups)
 - #164110 (callconv: mips64: Clarify why padding floats with integer args is correct)
 - #164120 (tests: Update duplicated-path-in-error message on musl hosts)
@rust-bors
rust-bors Bot merged commit f77c964 into rust-lang:main Oct 11, 2026
14 checks passed
@rustbot rustbot added this to the 1.101.0 milestone Oct 11, 2026
rust-bors Bot pushed a commit that referenced this pull request Oct 11, 2026
Rollup merge of #164020 - tmiasko:deduce-unit, r=jackh726

deduced_param_attrs for unit type offers no benefit

Outputs, where unit type is implicit default, did participate in the deduction. Identify unit type as irrelevant for the deduction as an optimization.
@tmiasko
tmiasko deleted the deduce-unit branch October 11, 2026 07:19
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