Skip to content

Conversation

beepster4096
Copy link
Contributor

@beepster4096 beepster4096 commented Oct 16, 2025

Fixes #147733 caused by #145513

This code in fact does not run only on runtime MIR.

@rustbot
Copy link
Collaborator

rustbot commented Oct 16, 2025

Some changes occurred to constck

cc @fee1-dead

Some changes occurred to the CTFE machinery

cc @RalfJung, @oli-obk, @lcnr

@rustbot rustbot 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 Oct 16, 2025
@rustbot
Copy link
Collaborator

rustbot commented Oct 16, 2025

r? @madsmtm

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

@madsmtm
Copy link
Contributor

madsmtm commented Oct 16, 2025

r? saethlin (since you reviewed the original PR)

@rustbot rustbot assigned saethlin and unassigned madsmtm Oct 16, 2025
@RalfJung
Copy link
Member

What kind of MIR does const_checks get run on, if it's not runtime MIR?

@beepster4096
Copy link
Contributor Author

It runs on post-cleanup analysis MIR right here right before the lowering to runtime MIR.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

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.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

ICE CopyForDeref in runtime MIR

5 participants