Skip to content

[OSLogOptimization] Improve SIL-level diagnostics generated for the os log APIs. #31120

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

Conversation

ravikandhadai
Copy link
Contributor

Now that there is already a Sema check for enforcing the user model, the SIL-level diagnostics can be a bit more targeted.

Also, suppress the errors generated by for the _globalStringTablePointerBuiltin when the OSLogOptimization pass emits diagnostics. This eliminates the cryptic error that always accompanies the diagnostics emitted by the OSLogOptimization pass.

os log APIs, based on the fact that there is already a Sema check for
enforcing the user model.

Suppress the errors generated by for the _globalStringTablePointerBuiltin
when the OSLogOptimization pass emits diagnostics. This elimiantes the
cryptic error that always accompanies the diagnostics emitted by the
OSLogOptimization pass.
@ravikandhadai
Copy link
Contributor Author

@swift-ci Please test

@swift-ci
Copy link
Contributor

Build failed
Swift Test Linux Platform
Git Sha - 2e4977f

@ravikandhadai
Copy link
Contributor Author

@swift-ci please smoke test Linux Platform

@ravikandhadai ravikandhadai merged commit dc42259 into swiftlang:master Apr 18, 2020
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