Skip to content

[5.9][IRGen] Reject enums with inaccessible tpye metadata in layout string… #65017

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

Merged
merged 4 commits into from
Apr 14, 2023

Conversation

drexin
Copy link
Contributor

@drexin drexin commented Apr 7, 2023

… generation

Cherry-picked from: #65016

rdar://107679697

Because we are currently handling most enums in layout strings by going through the metadata, we have to ensure that the metadata is accessible from the current module and reject the enum otherwise.

… generation

rdar://107679697

Because we are currently handling most enums in layout strings by going through the metadata, we have to ensure that the metadata is accessible from the current module and reject the enum otherwise.
@drexin drexin requested a review from a team as a code owner April 7, 2023 20:52
@drexin
Copy link
Contributor Author

drexin commented Apr 7, 2023

@swift-ci test

@drexin drexin changed the title [IRGen] Reject enums with inaccessible tpye metadata in layout string… [5.9][IRGen] Reject enums with inaccessible tpye metadata in layout string… Apr 7, 2023
@drexin
Copy link
Contributor Author

drexin commented Apr 8, 2023

@swift-ci test

@drexin
Copy link
Contributor Author

drexin commented Apr 13, 2023

@swift-ci test

@drexin
Copy link
Contributor Author

drexin commented Apr 14, 2023

@tbkka The PR to main was reviewed and approved by @aschwaighofer. Would you mind approving this?

@drexin drexin merged commit 993046e into swiftlang:release/5.9 Apr 14, 2023
@drexin drexin deleted the wip-107679697-5.9 branch April 14, 2023 15:02
@AnthonyLatsis AnthonyLatsis added the 🍒 release cherry pick Flag: Release branch cherry picks label May 3, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🍒 release cherry pick Flag: Release branch cherry picks swift 5.9
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants