Skip to content

[Foundation] adjust inline of append and initialization functions #17121

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

Conversation

phausler
Copy link
Contributor

Use _copyContents(initialzing:) to avoid performance regressions while using generic collection/sequence APIs. This was a regression that was introduced on by ABI stability hardening of Data on the 4.2 and master branches.

phausler added 2 commits June 11, 2018 15:10
… use _copyContents(initialzing:) to avoid performance regressions while using generic collection/sequence APIs
@phausler
Copy link
Contributor Author

@swift-ci please smoke test

@phausler
Copy link
Contributor Author

@shahmishal do you know what happened here? I branched this from the swift-4.2-branch and it is merge targeting to that branch.

@shahmishal
Copy link
Member

@swift-ci Please smoke test Linux

@phausler
Copy link
Contributor Author

@parkera I’m pretty sure this belongs in 4.2: do we have to pick this to other branches?

@parkera
Copy link
Contributor

parkera commented Jun 13, 2018

I think master and swift-4.2-branch are the two required right now.

@phausler
Copy link
Contributor Author

@swift-ci please smoke test and merge

2 similar comments
@phausler
Copy link
Contributor Author

@swift-ci please smoke test and merge

@phausler
Copy link
Contributor Author

@swift-ci please smoke test and merge

@airspeedswift
Copy link
Member

I think 4.2 needs full test before merge.

@phausler
Copy link
Contributor Author

@swift-ci please test and merge

@swift-ci swift-ci merged commit 25b98a1 into swiftlang:swift-4.2-branch Jun 14, 2018
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.

5 participants