-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[stdlib] Resolve unnecessary buffer copy in HashedCollections #6211
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
@swift-ci please test |
@airspeedswift Is this ready to merge? |
@gankro can you have a quick review since you were the last one looking at hashed collections? LGTM. |
@swift-ci please benchmark |
Build comment file:Build failed before running benchmark. |
@swift-ci please bench |
The change looks good to me, I just want to verify that there isn't any weird perf voodoo. |
@swift-ci please benchmark |
Build comment file:Build failed before running benchmark. |
Guess there is a problem with the CI. |
@swift-ci please benchmark |
(maintenance just finished) |
Build comment file:Optimized (O) Regression (0)Improvement (0)No Changes (147)
Regression (0)Improvement (0)No Changes (147)
|
Thanks! |
This patch creates a new buffer instead of removing items if the buffer is not uniquely referenced
Applied fix suggested in FIXME and ran tests locally