Skip to content
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

Remove special handling of the -experimental-lazy-typecheck flag #1468

Merged
merged 2 commits into from
Oct 19, 2023

Conversation

tshortli
Copy link
Contributor

@tshortli tshortli commented Oct 18, 2023

For now, the flag will simply be a frontend flag and the build system will pass it directly using -Xfrontend when appropriate.

Resolves rdar://117168788

@tshortli tshortli requested a review from artemcm October 18, 2023 22:35
@tshortli
Copy link
Contributor Author

@swift-ci please test

@tshortli
Copy link
Contributor Author

Related to swiftlang/swift#69259.

@tshortli
Copy link
Contributor Author

@swift-ci please test

@tshortli
Copy link
Contributor Author

@swift-ci please test Windows

For now, the flag will simply be a frontend flag and the build system will pass
it directly using `-Xfrontend` when appropriate.

Resolves rdar://117168788
@tshortli tshortli force-pushed the lazy-typecheck-emit-module-only branch from cb49485 to dd61121 Compare October 19, 2023 07:15
@tshortli tshortli changed the title Avoid passing -experimental-lazy-typecheck to compile jobs Remove special handling of the -experimental-lazy-typecheck flag Oct 19, 2023
@tshortli tshortli force-pushed the lazy-typecheck-emit-module-only branch from dd61121 to 98f1231 Compare October 19, 2023 07:20
@tshortli
Copy link
Contributor Author

@swift-ci please test

@tshortli
Copy link
Contributor Author

@swift-ci please test Windows

@tshortli
Copy link
Contributor Author

@swift-ci please test Windows

@tshortli tshortli merged commit 10d702f into swiftlang:main Oct 19, 2023
3 checks passed
@tshortli tshortli deleted the lazy-typecheck-emit-module-only branch October 19, 2023 21:46
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.

2 participants