Skip to content

[gardening] Remove uses of "importDeclCached" and cleanup "VisitEnumConstantDecl". #34956

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 1 commit into from
Dec 5, 2020

Conversation

zoecarver
Copy link
Contributor

Cleans up the EnumConstantDecl visitor by consolidating cases and removing calls to "importDeclCached".

There are many cleanups that need to happen (both in the design and QoI) for enums, but this is a start.

…onstantDecl".

Cleans up the EnumConstantDecl visitor by consolidating cases and
removing calls to "importDeclCached".
@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test.

3 similar comments
@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test.

@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test.

@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test.

@slavapestov
Copy link
Contributor

@swift-ci Please test source compatibility

@zoecarver
Copy link
Contributor Author

Thanks @slavapestov!

@zoecarver
Copy link
Contributor Author

Looks like somehow the CI got messed up. It looks like the Linux and MacOS bots both succeeded but aren't getting reported properly. I'll re-trigger them.

@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test Linux.

@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test OS X.

1 similar comment
@zoecarver
Copy link
Contributor Author

@swift-ci please smoke test OS X.

@zoecarver zoecarver merged commit 8104d5f into swiftlang:main Dec 5, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants