Skip to content

Do not ICE on private type in field of unresolved struct - #145369

Merged
bors merged 1 commit into
rust-lang:masterfrom
estebank:issue-145367
Aug 15, 2025
Merged

Do not ICE on private type in field of unresolved struct#145369
bors merged 1 commit into
rust-lang:masterfrom
estebank:issue-145367

Conversation

@estebank

@estebankestebank commented Aug 13, 2025•
edited
Loading

Copy link
Copy Markdown
Contributor

@rustbot

rustbot commented Aug 13, 2025

Copy link
Copy Markdown
Collaborator

r? @fee1-dead

rustbot has assigned @fee1-dead.
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

@rustbotrustbot 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 Aug 13, 2025
@rust-log-analyzer

This comment has been minimized.

Sign in to view
compiler-errors
compiler-errors reviewed Aug 13, 2025
View reviewed changes
Comment threadcompiler/rustc_resolve/src/diagnostics.rs Outdated
@compiler-errors

compiler-errors commented Aug 14, 2025

Copy link
Copy Markdown
Contributor

@bors r+ rollup

@bors

bors commented Aug 14, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit caadc8d has been approved by compiler-errors

It is now in the queue for this repository.

@borsbors 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 Aug 14, 2025
Zalathar added a commit to Zalathar/rust that referenced this pull request Aug 15, 2025
@Zalathar
Rollup merge ofrust-lang#145369- estebank:issue-145367, r=compiler-…
5f0612e
…errors
Do not ICE on private type in field of unresolved struct
Fixrust-lang#145367.
@ZalatharZalathar mentioned this pull request Aug 15, 2025
Closed
bors added a commit that referenced this pull request Aug 15, 2025
@bors
Auto merge of#145418- Zalathar:rollup-qgoixs3, r=Zalathar
f4b87c1
Rollup of 22 pull requests
Successful merges:
- #118087 (Add Ref/RefMut try_map method)
- #122661 (Change the desugaring of `assert!` for better error output)
- #140740 (Add `-Zindirect-branch-cs-prefix`)
- #142640 (Implement autodiff using intrinsics)
- #143075 (compiler: Allow `extern "interrupt" fn() -> !`)
- #144865 (Fix tail calls to `#[track_caller]` functions)
- #144944 (E0793: Clarify that it applies to unions as well)
- #144947 (Fix description of unsigned `checked_exact_div`)
- #145004 (Couple of minor cleanups)
- #145005 (strip prefix of temporary file names when it exceeds filesystem name length limit)
- #145012 (Tail call diagnostics to include lifetime info)
- #145065 (resolve: Introduce `RibKind::Block`)
- #145120 (llvm: Accept new LLVM lifetime format)
- #145189 (Weekly `cargo update`)
- #145235 (Minor `[const]` tweaks)
- #145275 (fix(compiler/rustc_codegen_llvm): apply `target-cpu` attribute)
- #145322 (Resolve the prelude import in `build_reduced_graph`)
- #145331 (Make std use the edition 2024 prelude)
- #145369 (Do not ICE on private type in field of unresolved struct)
- #145378 (Add `FnContext` in parser for diagnostic)
- #145389 ([rustdoc] Revert "rustdoc search: prefer stable items in search results")
- #145392 (coverage: Remove intermediate data structures from mapping creation)
r? `@ghost`
`@rustbot` modify labels: rollup
@ZalatharZalathar mentioned this pull request Aug 15, 2025
Merged
bors added a commit that referenced this pull request Aug 15, 2025
@bors
Auto merge of#145423- Zalathar:rollup-9jtefpl, r=Zalathar
ba412a6
Rollup of 21 pull requests
Successful merges:
- #118087 (Add Ref/RefMut try_map method)
- #122661 (Change the desugaring of `assert!` for better error output)
- #142640 (Implement autodiff using intrinsics)
- #143075 (compiler: Allow `extern "interrupt" fn() -> !`)
- #144865 (Fix tail calls to `#[track_caller]` functions)
- #144944 (E0793: Clarify that it applies to unions as well)
- #144947 (Fix description of unsigned `checked_exact_div`)
- #145004 (Couple of minor cleanups)
- #145005 (strip prefix of temporary file names when it exceeds filesystem name length limit)
- #145012 (Tail call diagnostics to include lifetime info)
- #145065 (resolve: Introduce `RibKind::Block`)
- #145120 (llvm: Accept new LLVM lifetime format)
- #145189 (Weekly `cargo update`)
- #145235 (Minor `[const]` tweaks)
- #145275 (fix(compiler/rustc_codegen_llvm): apply `target-cpu` attribute)
- #145322 (Resolve the prelude import in `build_reduced_graph`)
- #145331 (Make std use the edition 2024 prelude)
- #145369 (Do not ICE on private type in field of unresolved struct)
- #145378 (Add `FnContext` in parser for diagnostic)
- #145389 ([rustdoc] Revert "rustdoc search: prefer stable items in search results")
- #145392 (coverage: Remove intermediate data structures from mapping creation)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 36515e7 into rust-lang:masterAug 15, 2025
10 checks passed
@rustbotrustbot added this to the 1.91.0 milestone Aug 15, 2025
rust-timer added a commit that referenced this pull request Aug 15, 2025
@rust-timer
Unrolled build for#145369
e98fe85
Rollup merge of #145369 - estebank:issue-145367, r=compiler-errors
Do not ICE on private type in field of unresolved struct
Fix#145367.
github-actionsBot pushed a commit to rust-lang/rustc-dev-guide that referenced this pull request Aug 18, 2025
@bors
Auto merge of #145423 - Zalathar:rollup-9jtefpl, r=Zalathar
2446ebf
Rollup of 21 pull requests
Successful merges:
- rust-lang/rust#118087 (Add Ref/RefMut try_map method)
- rust-lang/rust#122661 (Change the desugaring of `assert!` for better error output)
- rust-lang/rust#142640 (Implement autodiff using intrinsics)
- rust-lang/rust#143075 (compiler: Allow `extern "interrupt" fn() -> !`)
- rust-lang/rust#144865 (Fix tail calls to `#[track_caller]` functions)
- rust-lang/rust#144944 (E0793: Clarify that it applies to unions as well)
- rust-lang/rust#144947 (Fix description of unsigned `checked_exact_div`)
- rust-lang/rust#145004 (Couple of minor cleanups)
- rust-lang/rust#145005 (strip prefix of temporary file names when it exceeds filesystem name length limit)
- rust-lang/rust#145012 (Tail call diagnostics to include lifetime info)
- rust-lang/rust#145065 (resolve: Introduce `RibKind::Block`)
- rust-lang/rust#145120 (llvm: Accept new LLVM lifetime format)
- rust-lang/rust#145189 (Weekly `cargo update`)
- rust-lang/rust#145235 (Minor `[const]` tweaks)
- rust-lang/rust#145275 (fix(compiler/rustc_codegen_llvm): apply `target-cpu` attribute)
- rust-lang/rust#145322 (Resolve the prelude import in `build_reduced_graph`)
- rust-lang/rust#145331 (Make std use the edition 2024 prelude)
- rust-lang/rust#145369 (Do not ICE on private type in field of unresolved struct)
- rust-lang/rust#145378 (Add `FnContext` in parser for diagnostic)
- rust-lang/rust#145389 ([rustdoc] Revert "rustdoc search: prefer stable items in search results")
- rust-lang/rust#145392 (coverage: Remove intermediate data structures from mapping creation)
r? `@ghost`
`@rustbot` modify labels: rollup
@github-actionsgithub-actionsBot mentioned this pull request Aug 18, 2025
Merged
Kobzol pushed a commit to Kobzol/rustc_codegen_gcc that referenced this pull request Dec 21, 2025
@bors
Auto merge of #145423 - Zalathar:rollup-9jtefpl, r=Zalathar
4640daf
Rollup of 21 pull requests
Successful merges:
- rust-lang/rust#118087 (Add Ref/RefMut try_map method)
- rust-lang/rust#122661 (Change the desugaring of `assert!` for better error output)
- rust-lang/rust#142640 (Implement autodiff using intrinsics)
- rust-lang/rust#143075 (compiler: Allow `extern "interrupt" fn() -> !`)
- rust-lang/rust#144865 (Fix tail calls to `#[track_caller]` functions)
- rust-lang/rust#144944 (E0793: Clarify that it applies to unions as well)
- rust-lang/rust#144947 (Fix description of unsigned `checked_exact_div`)
- rust-lang/rust#145004 (Couple of minor cleanups)
- rust-lang/rust#145005 (strip prefix of temporary file names when it exceeds filesystem name length limit)
- rust-lang/rust#145012 (Tail call diagnostics to include lifetime info)
- rust-lang/rust#145065 (resolve: Introduce `RibKind::Block`)
- rust-lang/rust#145120 (llvm: Accept new LLVM lifetime format)
- rust-lang/rust#145189 (Weekly `cargo update`)
- rust-lang/rust#145235 (Minor `[const]` tweaks)
- rust-lang/rust#145275 (fix(compiler/rustc_codegen_llvm): apply `target-cpu` attribute)
- rust-lang/rust#145322 (Resolve the prelude import in `build_reduced_graph`)
- rust-lang/rust#145331 (Make std use the edition 2024 prelude)
- rust-lang/rust#145369 (Do not ICE on private type in field of unresolved struct)
- rust-lang/rust#145378 (Add `FnContext` in parser for diagnostic)
- rust-lang/rust#145389 ([rustdoc] Revert "rustdoc search: prefer stable items in search results")
- rust-lang/rust#145392 (coverage: Remove intermediate data structures from mapping creation)
r? `@ghost`
`@rustbot` modify labels: rollup
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer
@compiler-errorscompiler-errorscompiler-errors left review comments
Reviewers whose approvals may not affect merge requirements

Assignees

@compiler-errorscompiler-errors

Labels

S-waiting-on-borsStatus: Waiting on bors to run and complete tests. Bors will change the label on completion.T-compilerRelevant to the compiler team, which will review and decide on the PR/issue.

Projects

None yet

Milestone

1.91.0

Development

Successfully merging this pull request may close these issues.

ICE: no entry found for key

6 participants

@estebank@rustbot@rust-log-analyzer@compiler-errors@bors@fee1-dead