Skip to content

[Distributed] avoid double typechecking dist func decl #42074

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
Mar 30, 2022

Conversation

ktoso
Copy link
Contributor

@ktoso ktoso commented Mar 29, 2022

Resolves rdar://90817485

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

@swift-ci please smoke test

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

@swift-ci please build toolchain macOS

@ktoso ktoso requested a review from DougGregor March 29, 2022 00:06
@ktoso ktoso added the distributed Feature → concurrency: distributed actor label Mar 29, 2022
@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

ok all good but just to be sure another full test run

@swift-ci please test and merge

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

Swift(watchsimulator-i386) :: Sema/spi-available-inline.swift

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

@swift-ci please build toolchain macOS

Need a toolchain to verify if this helped resolve the radar.

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

CI fixed in #42075

@swift-ci please build toolchain macOS

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

mac CI hanged... Cannot contact macos-node-i-018d7e169fd77215d: java.lang.InterruptedException

@ktoso
Copy link
Contributor Author

ktoso commented Mar 29, 2022

@swift-ci please test macOS

@ktoso ktoso merged commit 0d100e6 into swiftlang:main Mar 30, 2022
@ktoso ktoso deleted the wip-avoid-double-typechecking branch March 30, 2022 01:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
distributed Feature → concurrency: distributed actor
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants