Skip to content

Fix ICE in struct construction with base expression #91538

New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Closed

Conversation

compiler-errors
Copy link
Member

For some reason we ICE if we try to check_expr within a failing check_expr_has_type_or_error. We don't need to do that inner check_expr, though, since we've already computed it, so just pass it along to the extend_err closure.

Let me know if we should continue investigating the can't compose [Borrow(Ref('_#0r, Not)) -> &Option<Bar>] and [Borrow(Ref('_#3r, Not)) -> &Option<Bar>] ICE in FnCtxt::apply_adjustments which is the underlying cause for this ICE.

Fixes #91502

@rust-highfive
Copy link
Contributor

r? @wesleywiser

(rust-highfive has picked a reviewer for you, use r? to override)

@rust-highfive rust-highfive added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Dec 4, 2021
@apiraino apiraino added the T-compiler Relevant to the compiler team, which will review and decide on the PR/issue. label Dec 9, 2021
@bors
Copy link
Collaborator

bors commented Dec 28, 2021

☔ The latest upstream changes (presumably #92329) made this pull request unmergeable. Please resolve the merge conflicts.

@compiler-errors compiler-errors deleted the issue-91502 branch April 7, 2022 04:36
# for free to join this conversation on GitHub. Already have an account? # 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.

struct update syntax causes ICE
5 participants