Apply MCP 1003 and move diagnostics.rs into its own module by GuillaumeGomez · Pull Request #158732 · rust-lang/rust

GuillaumeGomez · 2026-07-03T13:08:12Z

Part of #158699.

r? @JonathanBrouwer

rustbot · 2026-07-03T13:08:15Z

Some changes occurred in exhaustiveness checking

cc @Nadrieril

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

Some changes occurred in compiler/rustc_attr_parsing

cc @jdonszelmann, @JonathanBrouwer

rustbot · 2026-07-03T13:08:18Z

JonathanBrouwer is currently at their maximum review capacity.
They may take a while to respond.

JonathanBrouwer · 2026-07-05T06:59:31Z

I don't think this is what the MCP said.
I think what we settled on is:

src/diagnostics.rs usually

src/diagnostics/mod.rs with src/diagnostics/impl.rs if there's a need for multiple files

View changes since the review

GuillaumeGomez · 2026-07-10T10:20:19Z

Finally made the only change for this PR before moving forward for the rest. :)

JonathanBrouwer

@bors r+ rollup

View changes since this review

rust-bors · 2026-07-10T10:41:47Z

📌 Commit 2de1207 has been approved by JonathanBrouwer

It is now in the queue for this repository.

@JonathanBrouwer

… r=JonathanBrouwer Apply MCP 1003 and move diagnostics.rs into its own module Fixes rust-lang#158699. r? @JonathanBrouwer

…uwer Rollup of 7 pull requests Successful merges: - #157690 (codegen_ssa: pack small const aggregates into immediate stores) - #159005 (Use `as_lang_item` instead of repeatedly matching) - #156735 (Move NativeLib::filename to the rmeta-link archive member) - #158732 (Apply MCP 1003 and move diagnostics.rs into its own module) - #158854 (Add `#[rustc_test_entrypoint_marker]`) - #158965 (Add codegen test for Result is_ok unwrap) - #159050 (assert only opaques with sub unified hidden infer are non-rigid)

@JonathanBrouwer

… r=JonathanBrouwer Apply MCP 1003 and move diagnostics.rs into its own module Fixes rust-lang#158699. r? @JonathanBrouwer

…uwer Rollup of 10 pull requests Successful merges: - #157690 (codegen_ssa: pack small const aggregates into immediate stores) - #157706 (Deny `todo!()` in tidy) - #159005 (Use `as_lang_item` instead of repeatedly matching) - #156735 (Move NativeLib::filename to the rmeta-link archive member) - #159039 (resolve: fix effective visibilities for items in ambiguous glob sets) - #158930 (Reorganize `tests/ui/issues` [20/N]) - #158965 (Add codegen test for Result is_ok unwrap) - #158979 (Reorganize `tests/ui/issues` [21/N]) - #159050 (assert only opaques with sub unified hidden infer are non-rigid) - #159062 (Remove unused WEAK_ONLY_LANG_ITEMS static) Failed merges: - #158732 (Apply MCP 1003 and move diagnostics.rs into its own module)

@JonathanBrouwer

… r=JonathanBrouwer Apply MCP 1003 and move diagnostics.rs into its own module Fixes rust-lang#158699. r? @JonathanBrouwer

…uwer Rollup of 12 pull requests Successful merges: - #157690 (codegen_ssa: pack small const aggregates into immediate stores) - #159005 (Use `as_lang_item` instead of repeatedly matching) - #156735 (Move NativeLib::filename to the rmeta-link archive member) - #157153 (allow `Allocator`s to be used as `#[global_allocator]`s) - #158767 (merge DefKind::InlineConst into AnonConst) - #159039 (resolve: fix effective visibilities for items in ambiguous glob sets) - #158732 (Apply MCP 1003 and move diagnostics.rs into its own module) - #158930 (Reorganize `tests/ui/issues` [20/N]) - #158965 (Add codegen test for Result is_ok unwrap) - #158979 (Reorganize `tests/ui/issues` [21/N]) - #159050 (assert only opaques with sub unified hidden infer are non-rigid) - #159062 (Remove unused WEAK_ONLY_LANG_ITEMS static)

@JonathanBrouwer

… r=JonathanBrouwer Apply MCP 1003 and move diagnostics.rs into its own module Part of rust-lang#158699. r? @JonathanBrouwer

@JonathanBrouwer

… r=JonathanBrouwer Apply MCP 1003 and move diagnostics.rs into its own module Part of rust-lang#158699. r? @JonathanBrouwer

GuillaumeGomez · 2026-07-11T16:03:34Z

Fixed merge conflicts.

@bors r=JonathanBrouwer

rust-bors · 2026-07-11T16:03:37Z

📌 Commit 63c6888 has been approved by JonathanBrouwer

It is now in the queue for this repository.

…uwer Rollup of 14 pull requests Successful merges: - #159126 (Fix PR number in bootstrap's change tracker) - #155811 (Include AtomicU128/AtomicI128 in docs for any target) - #156968 (diagnostics: suggest generic_const_args for const ops) - #159012 (Shrink mir::Statement to 40 bytes) - #156618 (rustdoc: test ignoring rustc lints in CLI) - #158182 (std: use `OnceLock` for SGX argument storage) - #159114 (Add regression test for $-prefixed fragment specifier in repetition) - #158523 (Fix feature gate for `repr(simd)`) - #158876 (Fix multiple logic bugs in `Arc::make_mut`) - #158928 (std: fix Xous UDP send_to length mismatch and truncation) - #159060 (Look for the cdb architecture that corresponds to the target triple) - #159089 (riscv: update c-variadic test for LLVM changes) - #159093 (pretty-print: use inline asm's actual macro name) - #159122 (doc: use ptr::addr in offset_from docs) Failed merges: - #158732 (Apply MCP 1003 and move diagnostics.rs into its own module)

… `error_helper.rs` into the `diagnostics` folder in `rustc_resolve`

rustbot · 2026-07-11T22:46:19Z

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.

GuillaumeGomez · 2026-07-11T22:46:27Z

Fixed merge conflicts v3.

@bors r=JonathanBrouwer

rust-bors · 2026-07-11T22:46:29Z

📌 Commit 4716c85 has been approved by JonathanBrouwer

It is now in the queue for this repository.

rustbot added A-attributes Area: Attributes (`#[…]`, `#![…]`) 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 Jul 3, 2026

rustbot assigned JonathanBrouwer Jul 3, 2026

petrochenkov reviewed Jul 3, 2026

View reviewed changes

Comment thread compiler/rustc_resolve/src/diagnostics/impls.rs

Comment thread compiler/rustc_resolve/src/late/diagnostics.rs

GuillaumeGomez force-pushed the rename-diagnostics branch from b8a763f to 6942310 Compare July 3, 2026 13:49

JonathanBrouwer reviewed Jul 5, 2026

View reviewed changes

JonathanBrouwer requested changes Jul 5, 2026

View reviewed changes

rustbot added S-waiting-on-author Status: This is awaiting some action (such as code changes or more information) from the author. and removed S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. labels Jul 5, 2026

This comment has been minimized.

Sign in to view

GuillaumeGomez force-pushed the rename-diagnostics branch from 6942310 to 2de1207 Compare July 10, 2026 10:19

This comment has been minimized.

Sign in to view

JonathanBrouwer approved these changes Jul 10, 2026

View reviewed changes

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 Jul 10, 2026

JonathanBrouwer mentioned this pull request Jul 10, 2026

Rollup of 7 pull requests #159068

Closed

This was referenced Jul 10, 2026

Rollup of 10 pull requests #159077

Closed

Rollup of 10 pull requests #159080

Closed

JonathanBrouwer mentioned this pull request Jul 10, 2026

Rollup of 12 pull requests #159095

Closed

GuillaumeGomez mentioned this pull request Jul 10, 2026

Apply MCP 1003 and move diagnostics.rs into its own module - part 2 #159096

Closed

JonathanBrouwer mentioned this pull request Jul 10, 2026

Rollup of 14 pull requests #159104

Closed

JonathanBrouwer mentioned this pull request Jul 10, 2026

Rollup of 15 pull requests #159107

Closed

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 Jul 10, 2026

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 Jul 10, 2026

GuillaumeGomez force-pushed the rename-diagnostics branch from 2de1207 to 63c6888 Compare July 11, 2026 16:03

This comment has been minimized.

Sign in to view

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 Jul 11, 2026

JonathanBrouwer mentioned this pull request Jul 11, 2026

Rollup of 14 pull requests #159142

Merged

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 Jul 11, 2026

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 Jul 11, 2026

Apply MCP 1003 and move diagnostics.rs into its own module and move…

4716c85

… `error_helper.rs` into the `diagnostics` folder in `rustc_resolve`

GuillaumeGomez force-pushed the rename-diagnostics branch from 63c6888 to 4716c85 Compare July 11, 2026 22:46

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 Jul 11, 2026

Uh oh!

Uh oh!

Conversation

GuillaumeGomez commented Jul 3, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Uh oh!

rustbot commented Jul 3, 2026

Uh oh!

rustbot commented Jul 3, 2026

Uh oh!

Uh oh!

Uh oh!

JonathanBrouwer Jul 5, 2026 • edited by rustbot 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.

This comment has been minimized.

GuillaumeGomez commented Jul 10, 2026

Uh oh!

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

Uh oh!

Choose a reason for hiding this comment

Uh oh!

rust-bors Bot commented Jul 10, 2026

Uh oh!

This comment has been minimized.

This comment has been minimized.

GuillaumeGomez commented Jul 11, 2026

Uh oh!

rust-bors Bot commented Jul 11, 2026

Uh oh!

This comment has been minimized.

rustbot commented Jul 11, 2026

Uh oh!

GuillaumeGomez commented Jul 11, 2026

Uh oh!

rust-bors Bot commented Jul 11, 2026

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

4 participants

GuillaumeGomez commented Jul 3, 2026 •

edited

Loading

JonathanBrouwer Jul 5, 2026 •

edited by rustbot

Loading

JonathanBrouwer left a comment •

edited by rustbot

Loading