Skip to content

AST, SIL: Remove UTF16 encoding from StringLiteralExpr and StringLiteralInst #33288

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
Aug 7, 2020

Conversation

eeckstein
Copy link
Contributor

The UTF16 encoding is not used (anymore). I think it became obsolete with the switch to the UTF8 String representation.

@eeckstein eeckstein requested a review from slavapestov August 4, 2020 18:58
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test

@eeckstein
Copy link
Contributor Author

@swift-ci smoke test linux

@eeckstein eeckstein requested a review from DougGregor August 5, 2020 09:07
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test macOS

…ralInst

The UTF16 encoding is not used (anymore). I think it became obsolete with the switch to the UTF8 String representation.
@eeckstein eeckstein force-pushed the string-optimization branch from 0d8e75d to 755f6aa Compare August 6, 2020 17:10
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test and merge

1 similar comment
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test and merge

@eeckstein
Copy link
Contributor Author

@swift-ci smoke test macOS

1 similar comment
@eeckstein
Copy link
Contributor Author

@swift-ci smoke test macOS

@eeckstein eeckstein merged commit ea30413 into swiftlang:master Aug 7, 2020
@eeckstein eeckstein deleted the string-optimization branch August 7, 2020 14:18
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