Remove the diagnostic lints #150879

JonathanBrouwer · 2026-01-09T15:22:31Z

Removes the untranslatable_diagnostic and diagnostic_outside_of_impl lints
These lints are allowed for a while already. Per rust-lang/compiler-team#959, we no longer want to enforce struct diagnostics for all usecases, so this is no longer useful.

r? @Kivooeo
I recommend reviewing commit by commit (also feel free to wait with reviewing until the MCP is accepted)

@rustbot +S-blocked
Blocked by rust-lang/compiler-team#959

rustbot · 2026-01-09T15:22:35Z

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

rust-analyzer is developed in its own repository. If possible, consider making this change to rust-lang/rust-analyzer instead.

cc @rust-lang/rust-analyzer

The Miri subtree was changed

cc @rust-lang/miri

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann

Some changes occurred in compiler/rustc_hir/src/attrs

cc @jdonszelmann

Some changes occurred in compiler/rustc_passes/src/check_attr.rs

cc @jdonszelmann

Some changes occurred to constck

cc @fee1-dead

Some changes occurred to the CTFE / Miri interpreter

cc @rust-lang/miri

Some changes occurred to the CTFE machinery

cc @RalfJung, @oli-obk, @lcnr

Some changes occurred in src/tools/clippy

cc @rust-lang/clippy

Some changes occurred in src/tools/rustfmt

cc @rust-lang/rustfmt

Some changes occurred in exhaustiveness checking

cc @Nadrieril

Some changes occurred in compiler/rustc_codegen_cranelift

cc @bjorn3

bjorn3 · 2026-01-09T15:34:42Z

As a nice side effect this will make patching rustc to support bootstrapping from rustc 1.83 and below a tiny bit easier. #[rustc_lint_diagnostics] was only introduced in rustc 1.84.

rust-bors · 2026-01-11T15:50:21Z

☔ The latest upstream changes (presumably #150957) made this pull request unmergeable. Please resolve the merge conflicts.

Kivooeo

r=me with resolved conflicts and mcp accepted

View changes since this review

JonathanBrouwer · 2026-01-19T14:44:52Z

@bors r=Kivooeo rollup

rust-bors · 2026-01-19T14:44:54Z

📌 Commit a545e0e has been approved by Kivooeo

It is now in the queue for this repository.

…e_diagnostic` throughout the codebase This PR was mostly made by search&replacing

…L` lints

rustbot · 2026-01-19T16:43:34Z

This PR was rebased onto a different main commit. Here's a range-diff highlighting what actually changed.

Rebasing is a normal part of keeping PRs up to date, so no action is needed—this note is just to help reviewers.

JonathanBrouwer · 2026-01-19T16:43:40Z

@bors r=Kivooeo rollup

rust-bors · 2026-01-19T16:43:43Z

📌 Commit 0fa5589 has been approved by Kivooeo

It is now in the queue for this repository.

@ghost

…uwer Rollup of 15 pull requests Successful merges: - #148623 (Ignore `#[doc(hidden)]` items when computing trimmed paths for printing) - #150550 (Miscellaneous cleanups to borrowck related code) - #150879 (Remove the diagnostic lints) - #150895 (rustc_errors: Add (heuristic) Syntax Highlighting for `rustc --explain`) - #150987 (remote-test-server: Fix header in batch mode) - #151004 (std: implement `sleep_until` on Apple platforms) - #151045 (Simplify some literal-value negations with `u128::wrapping_neg`) - #151119 (Support pointers in type reflection) - #151171 (Do not recover from `Trait()` if generic list is unterminated) - #151231 (HIR typeck cleanup: clarify and re-style `check_expr_unop`) - #151249 (Parse ident with allowing recovery when trying to diagnose) - #151295 (THIR patterns: Use `ty::Value` in more places throughout `const_to_pat`) - #151326 (Remove `DiagMessage::Translated` in favour of `DiagMessage::Str`) - #151361 (add test for issue 61463) - #151371 (Add `S-blocked` to `labels_blocking_approval`) r? @ghost

@Kivooeo

Rollup merge of #150879 - remove_diag_lints, r=Kivooeo Remove the diagnostic lints Removes the `untranslatable_diagnostic` and `diagnostic_outside_of_impl` lints These lints are allowed for a while already. Per rust-lang/compiler-team#959, we no longer want to enforce struct diagnostics for all usecases, so this is no longer useful. r? @Kivooeo I recommend reviewing commit by commit (also feel free to wait with reviewing until the MCP is accepted) @rustbot +S-blocked Blocked by rust-lang/compiler-team#959

@ghost

…uwer Rollup of 15 pull requests Successful merges: - rust-lang/rust#148623 (Ignore `#[doc(hidden)]` items when computing trimmed paths for printing) - rust-lang/rust#150550 (Miscellaneous cleanups to borrowck related code) - rust-lang/rust#150879 (Remove the diagnostic lints) - rust-lang/rust#150895 (rustc_errors: Add (heuristic) Syntax Highlighting for `rustc --explain`) - rust-lang/rust#150987 (remote-test-server: Fix header in batch mode) - rust-lang/rust#151004 (std: implement `sleep_until` on Apple platforms) - rust-lang/rust#151045 (Simplify some literal-value negations with `u128::wrapping_neg`) - rust-lang/rust#151119 (Support pointers in type reflection) - rust-lang/rust#151171 (Do not recover from `Trait()` if generic list is unterminated) - rust-lang/rust#151231 (HIR typeck cleanup: clarify and re-style `check_expr_unop`) - rust-lang/rust#151249 (Parse ident with allowing recovery when trying to diagnose) - rust-lang/rust#151295 (THIR patterns: Use `ty::Value` in more places throughout `const_to_pat`) - rust-lang/rust#151326 (Remove `DiagMessage::Translated` in favour of `DiagMessage::Str`) - rust-lang/rust#151361 (add test for issue 61463) - rust-lang/rust#151371 (Add `S-blocked` to `labels_blocking_approval`) r? @ghost

Kobzol · 2026-01-20T08:21:53Z

@rust-timer build d14532d

Checking for #151381.

rust-timer · 2026-01-20T10:23:57Z

Finished benchmarking commit (d14532d): comparison URL.

Overall result: ❌ regressions - no action needed

Benchmarking this pull request means it may be perf-sensitive – we'll automatically label it not fit for rolling up. You can override this, but we strongly advise not to, due to possible changes in compiler perf.

@bors rollup=never
@rustbot label: -S-waiting-on-perf -perf-regression

Instruction count

Our most reliable metric. Used to determine the overall result above. However, even this metric can be noisy.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	1.1%	[1.1%, 1.1%]	1
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-	-	0

Max RSS (memory usage)

Results (primary -2.1%, secondary 2.6%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	2.6%	[1.1%, 4.0%]	2
Improvements ✅ (primary)	-2.1%	[-2.4%, -1.8%]	2
Improvements ✅ (secondary)	-	-	0
All ❌✅ (primary)	-2.1%	[-2.4%, -1.8%]	2

Cycles

Results (secondary -2.5%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-	-	0
Improvements ✅ (secondary)	-2.5%	[-2.5%, -2.5%]	1
All ❌✅ (primary)	-	-	0

Binary size

Results (primary -0.0%, secondary -0.1%)

A less reliable metric. May be of interest, but not used to determine the overall result above.

	mean	range	count
Regressions ❌ (primary)	-	-	0
Regressions ❌ (secondary)	-	-	0
Improvements ✅ (primary)	-0.0%	[-0.0%, -0.0%]	3
Improvements ✅ (secondary)	-0.1%	[-0.1%, -0.1%]	4
All ❌✅ (primary)	-0.0%	[-0.0%, -0.0%]	3

Bootstrap: 473.058s -> 473.396s (0.07%)
Artifact size: 383.29 MiB -> 383.28 MiB (-0.00%)

rustbot assigned Kivooeo Jan 9, 2026

JonathanBrouwer added the S-blocked Status: Blocked on something else such as an RFC or other implementation work. label Jan 9, 2026

This comment has been minimized.

Sign in to view

JonathanBrouwer force-pushed the remove_diag_lints branch from 7be7f33 to 5889e9e Compare January 9, 2026 16:04

Kivooeo approved these changes Jan 18, 2026

View reviewed changes

JonathanBrouwer mentioned this pull request Jan 19, 2026

Tracking issue for MCP 959: Remove the fluent files #151366

Open

5 tasks

JonathanBrouwer force-pushed the remove_diag_lints branch from 5889e9e to a545e0e Compare January 19, 2026 13:35

This comment has been minimized.

Sign in to view

rust-bors bot removed the S-waiting-on-bors Status: Waiting on bors to run and complete tests. Bors will change the label on completion. label Jan 19, 2026

JonathanBrouwer added 4 commits January 19, 2026 17:39

Remove all allows for diagnostic_outside_of_impl and `untranslatabl…

0ee7d96

…e_diagnostic` throughout the codebase This PR was mostly made by search&replacing

Remove the UNTRANSLATABLE_DIAGNOSTIC and `DIAGNOSTIC_OUTSIDE_OF_IMP…

14484ec

…L` lints

Remove all usages of #[rustc_lint_diagnostics]

f50f5dc

Remove #[rustc_lint_diagnostics]

b7f6e85

Remove obsolote diagnostic tests

0fa5589

JonathanBrouwer force-pushed the remove_diag_lints branch from a545e0e to 0fa5589 Compare January 19, 2026 16:43

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-author Status: This is awaiting some action (such as code changes or more information) from the author. labels Jan 19, 2026

JonathanBrouwer mentioned this pull request Jan 19, 2026

Rollup of 15 pull requests #151381

Merged

rust-bors bot merged commit 958d1f9 into rust-lang:main Jan 20, 2026
11 checks passed

rustbot added this to the 1.95.0 milestone Jan 20, 2026

This comment has been minimized.

Sign in to view

Uh oh!

Remove the diagnostic lints #150879

Remove the diagnostic lints #150879

Conversation

JonathanBrouwer commented Jan 9, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rustbot commented Jan 9, 2026

Uh oh!

bjorn3 commented Jan 9, 2026

Uh oh!

This comment has been minimized.

rust-bors bot commented Jan 11, 2026

Uh oh!

Kivooeo left a comment • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Choose a reason for hiding this comment

Uh oh!

This comment has been minimized.

JonathanBrouwer commented Jan 19, 2026

Uh oh!

rust-bors bot commented Jan 19, 2026

Uh oh!

This comment has been minimized.

rustbot commented Jan 19, 2026

Uh oh!

JonathanBrouwer commented Jan 19, 2026

Uh oh!

rust-bors bot commented Jan 19, 2026

Uh oh!

Uh oh!

Kobzol commented Jan 20, 2026

Uh oh!

This comment has been minimized.

rust-timer commented Jan 20, 2026

Overall result: ❌ regressions - no action needed

Instruction count

Max RSS (memory usage)

Cycles

Binary size

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

7 participants

JonathanBrouwer commented Jan 9, 2026 •

edited

Loading

Kivooeo left a comment •

edited

Loading