Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Uplift generator movability to rustc_type_ir #116918

Closed
wants to merge 1 commit into from

Conversation

compiler-errors
Copy link
Member

Part of rust-lang/types-team#124

r? types

@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. WG-trait-system-refactor The Rustc Trait System Refactor Initiative labels Oct 19, 2023
@rustbot
Copy link
Collaborator

rustbot commented Oct 19, 2023

This PR changes Stable MIR

cc @oli-obk, @celinval, @spastorino, @ouz-a

This PR changes MIR

cc @oli-obk, @RalfJung, @JakobDegen, @davidtwco, @celinval, @vakaras

Some changes occurred to the core trait solver

cc @rust-lang/initiative-trait-system-refactor

Some changes occurred to MIR optimizations

cc @rust-lang/wg-mir-opt

@oli-obk
Copy link
Contributor

oli-obk commented Oct 19, 2023

Can we somehow keep sharing the hir and ir Movability type? Or is there no existing crate dependency from rustc_hir to rustc_ir?

@RalfJung
Copy link
Member

Why are there two of these PRs...? This an #116919.

@compiler-errors
Copy link
Member Author

compiler-errors commented Oct 19, 2023

@RalfJung because they're different types and are pretty different review sizes.

This PR touches 19 files, mostly within the compiler. That PR touches 124 files which span across the compiler, codegen backends, clippy, miri, etc.

@compiler-errors
Copy link
Member Author

@oli-obk: No, there are no crate deps between them that would make sense to put this type in.

@RalfJung
Copy link
Member

Oh, movability != mutability. 🤦

@compiler-errors compiler-errors changed the title Uplift movability to rustc_type_ir Uplift generator movability to rustc_type_ir Oct 19, 2023
@oli-obk
Copy link
Contributor

oli-obk commented Oct 19, 2023

@oli-obk: No, there are no crate deps between them that would make sense to put this type in.

If this becomes more common, maybe we should create such a crate. We merged some of these types to avoid any conversions or import confusions

@bors
Copy link
Contributor

bors commented Oct 19, 2023

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

@lcnr
Copy link
Contributor

lcnr commented Oct 19, 2023

It feels like creating a new rustc_ir_core or whatever for Mutability would significantly help with #116919 🤔 so I would prefer this over having duplicate types.

THough alternatively, can rustc_hir just depend on rustc_type_ir? This feels like the easiest solution here? 🤔

@compiler-errors compiler-errors deleted the movability branch December 21, 2023 01:53
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. WG-trait-system-refactor The Rustc Trait System Refactor Initiative
Projects
None yet
Development

Successfully merging this pull request may close these issues.

6 participants