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

Frontend: Reinstate suppression of warnings during interface type-checking #80415

Conversation

tshortli
Copy link
Contributor

We're not ready to start emitting warnings when type-checking modules from interface. It is causing performance regressions and spurious diagnostics to be emitted.

Reverts a small part of #80360.

Resolves rdar://148257136.

@tshortli tshortli requested a review from beccadax March 31, 2025 17:49
@tshortli tshortli requested a review from artemcm as a code owner March 31, 2025 17:49
@tshortli tshortli enabled auto-merge March 31, 2025 17:49
@tshortli tshortli force-pushed the suppress-warnings-when-typechecking-module-interfaces branch from 994ac78 to ec6acfd Compare March 31, 2025 18:21
@tshortli
Copy link
Contributor Author

@swift-ci please smoke test

@tshortli
Copy link
Contributor Author

@swift-ci please smoke test Linux

@tshortli
Copy link
Contributor Author

@swift-ci please smoke test Windows

…cking.

We're not ready to start emitting warnings when type-checking modules from
interface. It is causing performance regressions and spurious diagnostics to be
emitted.

Reverts a small part of swiftlang#80360.

Resolves rdar://148257136.
@tshortli tshortli force-pushed the suppress-warnings-when-typechecking-module-interfaces branch from ec6acfd to 645628f Compare April 1, 2025 03:41
@tshortli
Copy link
Contributor Author

tshortli commented Apr 1, 2025

@swift-ci please smoke test

@tshortli tshortli requested a review from beccadax April 1, 2025 03:42
Copy link
Contributor

@beccadax beccadax left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great!

@tshortli tshortli merged commit 8c8ed34 into swiftlang:main Apr 1, 2025
3 checks passed
@tshortli tshortli deleted the suppress-warnings-when-typechecking-module-interfaces branch April 1, 2025 13:44
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.

3 participants