Skip to content

Conversation

samueltardieu
Copy link
Member

@samueltardieu samueltardieu commented Aug 7, 2025

A type with escaping bound vars cannot be wrapped in a dummy binder during size computation.

Fixes #15429

changelog: [zero_sized_hashmap_values]: fix ICE in types with escaping lifetimes

r? Jarcho

Summary Notes

Managed by @rustbot—see help for details

@samueltardieu samueltardieu added the I-ICE Issue: Clippy panicked, giving an Internal Compilation Error (ICE) ❄️ label Aug 7, 2025
@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 7, 2025
@samueltardieu
Copy link
Member Author

samueltardieu commented Aug 7, 2025

@rustbot note Beta nomination
@rustbot label +beta-nominated

The bug is a regression from stable to stable in Clippy 1.89. The fix should be applied to 1.90, and probably to 1.89.1 if there is a point release.

The fix is safe as it makes the lint triggers in less cases. The only risk, and it is unlikely, would be the creation of false negatives which would be benign.

Cc @flip1995

@rustbot rustbot added the beta-nominated Nominated for backporting to the compiler in the beta channel. label Aug 7, 2025
A type with escaping bound vars cannot be wrapped in a dummy binder
during size computation.
@samueltardieu
Copy link
Member Author

Force-pushed to update comment with a more precise one.

Copy link
Contributor

@Jarcho Jarcho left a comment

Choose a reason for hiding this comment

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

Thank you.

@Jarcho Jarcho added this pull request to the merge queue Aug 9, 2025
Merged via the queue into rust-lang:master with commit ea7ebaa Aug 9, 2025
13 of 15 checks passed
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties label Aug 9, 2025
@samueltardieu samueltardieu deleted the issue-15429 branch August 9, 2025 21:52
@flip1995 flip1995 added beta-accepted Accepted for backporting to the compiler in the beta channel. and removed beta-nominated Nominated for backporting to the compiler in the beta channel. labels Sep 11, 2025
@flip1995
Copy link
Member

rust-lang/rust#146438

bors pushed a commit to rust-lang/rust that referenced this pull request Sep 11, 2025
)

A type with escaping bound vars cannot be wrapped in a dummy binder
during size computation.

Fixes rust-lang/rust-clippy#15429

changelog: [`zero_sized_hashmap_values`]: fix ICE in types with escaping
lifetimes

r? Jarcho

<!-- TRIAGEBOT_START -->

<!-- TRIAGEBOT_SUMMARY_START -->

### Summary Notes

- [Beta
nomination](rust-lang/rust-clippy#15434 (comment))
by [samueltardieu](https://github.com/samueltardieu)

*Managed by `@rustbot`—see
[help](https://forge.rust-lang.org/triagebot/note.html) for details*

<!-- TRIAGEBOT_SUMMARY_END -->
<!-- TRIAGEBOT_END -->
bors added a commit to rust-lang/rust that referenced this pull request Sep 11, 2025
[beta] Clippy backport

Only a single commit to backport:

- rust-lang/rust-clippy#15434

We'd like to get this ICE fix in a release earlier.

r? Mark-Simulacrum
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
beta-accepted Accepted for backporting to the compiler in the beta channel. I-ICE Issue: Clippy panicked, giving an Internal Compilation Error (ICE) ❄️
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Internal compiler error after upgrading to 1.89.0
4 participants