Skip to content

Don't ICE on non-TypeId metadata within TypeId - #144256

Merged
bors merged 1 commit into
rust-lang:masterfrom
oli-obk:type-id-ice
Jul 23, 2025
Merged

Don't ICE on non-TypeId metadata within TypeId#144256
bors merged 1 commit into
rust-lang:masterfrom
oli-obk:type-id-ice

Conversation

@oli-obk

@oli-obkoli-obk commented Jul 21, 2025

Copy link
Copy Markdown
Contributor

@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 Jul 21, 2025
@rustbot

Copy link
Copy Markdown
Collaborator

Some changes occurred to the CTFE / Miri interpreter

cc @rust-lang/miri

Some changes occurred to the CTFE machinery

cc @RalfJung, @oli-obk, @lcnr

@RalfJung

Copy link
Copy Markdown
Member

Makes sense! r=me when CI is green.

@rust-log-analyzer

This comment has been minimized.

@oli-obk

Copy link
Copy Markdown
ContributorAuthor

@bors r=RalfJung

@bors

bors commented Jul 22, 2025

Copy link
Copy Markdown
Collaborator

📌 Commit b2f8b40 has been approved by RalfJung

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 Jul 22, 2025
@oli-obk

Copy link
Copy Markdown
ContributorAuthor

@bors rollup

matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jul 22, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? `@RalfJung`
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jul 22, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? ``@RalfJung``
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jul 22, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? ```@RalfJung```
bors added a commit that referenced this pull request Jul 22, 2025
Rollup of 8 pull requests
Successful merges:
- #144094 (Ensure we codegen the main fn)
- #144173 (Remove tidy checks for `tests/ui/issues/`)
- #144218 (Use serde for target spec json deserialize)
- #144221 (generate elf symbol version in raw-dylib)
- #144234 (Fix broken TLS destructors on 32-bit win7)
- #144256 (Don't ICE on non-TypeId metadata within TypeId)
- #144272 (resolve: Make disambiguators for underscore bindings module-local (take 2))
- #144276 (Use less HIR in check_private_in_public.)
r? `@ghost`
`@rustbot` modify labels: rollup
jhpratt added a commit to jhpratt/rust that referenced this pull request Jul 23, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? ````@RalfJung````
@jhprattjhpratt mentioned this pull request Jul 23, 2025
jhpratt added a commit to jhpratt/rust that referenced this pull request Jul 23, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? `````@RalfJung`````
@jhprattjhpratt mentioned this pull request Jul 23, 2025
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jul 23, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? ``````@RalfJung``````
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jul 23, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? ```````@RalfJung```````
matthiaskrgr added a commit to matthiaskrgr/rust that referenced this pull request Jul 23, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? ````````@RalfJung````````
bors added a commit that referenced this pull request Jul 23, 2025
Rollup of 10 pull requests
Successful merges:
- #144173 (Remove tidy checks for `tests/ui/issues/`)
- #144234 (Fix broken TLS destructors on 32-bit win7)
- #144239 (Clean `rustc/parse/src/lexer` to improve maintainability)
- #144247 (coretests/num: use ldexp instead of hard-coding a power of 2)
- #144256 (Don't ICE on non-TypeId metadata within TypeId)
- #144290 (update tests/ui/SUMMARY.md)
- #144292 (mbe: Use concrete type for `get_unused_rule`)
- #144298 (coverage: Enlarge empty spans during MIR instrumentation, not codegen)
- #144311 (Add powerpc64le-unknown-linux-musl to CI rustc targets)
- #144315 (bootstrap: add package.json and package-lock.json to dist tarball)
r? `@ghost`
`@rustbot` modify labels: rollup
GuillaumeGomez added a commit to GuillaumeGomez/rust that referenced this pull request Jul 23, 2025
Don't ICE on non-TypeId metadata within TypeId
fixesrust-lang#144253
r? `````````@RalfJung`````````
bors added a commit that referenced this pull request Jul 23, 2025
Rollup of 10 pull requests
Successful merges:
- #144173 (Remove tidy checks for `tests/ui/issues/`)
- #144234 (Fix broken TLS destructors on 32-bit win7)
- #144239 (Clean `rustc/parse/src/lexer` to improve maintainability)
- #144247 (coretests/num: use ldexp instead of hard-coding a power of 2)
- #144256 (Don't ICE on non-TypeId metadata within TypeId)
- #144290 (update tests/ui/SUMMARY.md)
- #144292 (mbe: Use concrete type for `get_unused_rule`)
- #144298 (coverage: Enlarge empty spans during MIR instrumentation, not codegen)
- #144311 (Add powerpc64le-unknown-linux-musl to CI rustc targets)
- #144315 (bootstrap: add package.json and package-lock.json to dist tarball)
r? `@ghost`
`@rustbot` modify labels: rollup
try-job: x86_64-gnu-aux
bors added a commit that referenced this pull request Jul 23, 2025
Rollup of 9 pull requests
Successful merges:
- #144173 (Remove tidy checks for `tests/ui/issues/`)
- #144234 (Fix broken TLS destructors on 32-bit win7)
- #144239 (Clean `rustc/parse/src/lexer` to improve maintainability)
- #144256 (Don't ICE on non-TypeId metadata within TypeId)
- #144290 (update tests/ui/SUMMARY.md)
- #144292 (mbe: Use concrete type for `get_unused_rule`)
- #144298 (coverage: Enlarge empty spans during MIR instrumentation, not codegen)
- #144311 (Add powerpc64le-unknown-linux-musl to CI rustc targets)
- #144315 (bootstrap: add package.json and package-lock.json to dist tarball)
r? `@ghost`
`@rustbot` modify labels: rollup
@bors
bors merged commit 16c10c9 into rust-lang:masterJul 23, 2025
11 checks passed
@rustbotrustbot added this to the 1.90.0 milestone Jul 23, 2025
rust-timer added a commit that referenced this pull request Jul 23, 2025
Rollup merge of #144256 - oli-obk:type-id-ice, r=RalfJung
Don't ICE on non-TypeId metadata within TypeId
fixes#144253
r? ``````````@RalfJung``````````
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

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

Development

Successfully merging this pull request may close these issues.

ICE from transmuting pointer to TypeId then comparing in const

5 participants

@oli-obk@rustbot@RalfJung@rust-log-analyzer@bors