Skip to content

Conversation

ShoyuVanilla
Copy link
Member

I guess this is the last thing for the errors like Foo and Foo have similar names, but are actually distinct types 🙃

We might need CI that runs cargo check with in-rust-tree feature and check whether we are using ra_ap_* things

@rustbot rustbot added the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 18, 2025
@ShoyuVanilla
Copy link
Member Author

BTW we have several other errors beside this that complaining we are missing a field in a constructor of a type in rustc_type_ir

@lnicola
Copy link
Member

lnicola commented Sep 18, 2025

How can that be? Don't tell me WithCachedTypeInfo changed in the past hour or so.

@ShoyuVanilla ShoyuVanilla added this pull request to the merge queue Sep 18, 2025
@lnicola lnicola changed the title fix: Fix one more thing in in-rust-tree minor: Fix one more thing in in-rust-tree Sep 18, 2025
Merged via the queue into rust-lang:master with commit 958a8d0 Sep 18, 2025
15 checks passed
@ShoyuVanilla ShoyuVanilla deleted the fix-another-in-tree branch September 18, 2025 19:45
@rustbot rustbot removed the S-waiting-on-review Status: Awaiting review from the assignee but also interested parties. label Sep 18, 2025
@lnicola
Copy link
Member

lnicola commented Sep 18, 2025

Oh, it's gated by #[cfg(feature = "nightly")]. We could probably use Fingerprint::ZERO, but we can't refer to it.

@ShoyuVanilla
Copy link
Member Author

Oh, it's gated by #[cfg(feature = "nightly")]. We could probably use Fingerprint::ZERO, but we can't refer to it.

Oh, we're in a tight spot

@lnicola
Copy link
Member

lnicola commented Sep 19, 2025

Nah, it's fine, thanks to your awesome extern crate rustc_data_structures as ena hack 😅.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants