Only prevent field projections into opaque types, not types containing opaque ty...
source link: https://github.com/rust-lang/rust/pull/116156
Go to the source link to view the article. You can view the picture content, updated content and better typesetting reading experience. If the link is broken, please click the button below to view the snapshot at that time.
Only prevent field projections into opaque types, not types containing opaque types #116156
Conversation
Contributor
fixes #115778
I did not think that original condition through properly... I'll also need to check the similar check around the other ProjectionKind::OpaqueCast
creation site (this one is in hir, the other one is in mir), but I'll do that change in another PR that doesn't go into a beta backport.
added the beta-nominated Nominated for backporting to the compiler in the beta channel. label
Collaborator
(rustbot has picked a reviewer for you, use r? to override) |
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
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
Contributor
☀️ Test successful - checks-actions |
Collaborator
Finished benchmarking commit (e1636a0): comparison URL. Overall result: no relevant changes - no action needed@rustbot label: -perf-regression Instruction countThis benchmark run did not return any relevant results for this metric. Max RSS (memory usage)Results CyclesThis benchmark run did not return any relevant results for this metric. Binary sizeThis benchmark run did not return any relevant results for this metric. Bootstrap: 631.857s -> 630.538s (-0.21%) |
added the beta-accepted Accepted for backporting to the compiler in the beta channel. label
removed the beta-nominated Nominated for backporting to the compiler in the beta channel. label
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
None yet
Successfully merging this pull request may close these issues.
Recommend
About Joyk
Aggregate valuable and interesting links.
Joyk means Joy of geeK