-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[stdlib] Implement customization points for StrideTo, StrideThrough, and LazyFilterX #12390
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 benchmark |
@swift-ci Please smoke test |
6c1c331
to
59ada5f
Compare
@swift-ci Please benchmark |
@swift-ci Please smoke test |
@swift-ci Please clean test Linux platform |
Build failed |
Build comment file:Optimized (O)Regression (3)
Improvement (16)
No Changes (314)
Unoptimized (Onone)Regression (1)
Improvement (4)
No Changes (328)
Hardware Overview
|
@swift-ci Please smoke benchmark |
Build comment file:Optimized (O)Regression (4)
Improvement (3)
No Changes (326)
Unoptimized (Onone)Regression (4)
Improvement (3)
No Changes (326)
Hardware Overview
|
…ment for StrideTo and StrideThrough
…nd LazyXCollection
@swift-ci Please smoke benchmark |
@swift-ci Please test |
Build failed |
Build failed |
Build comment file:Optimized (O)Regression (5)
Improvement (15)
No Changes (314)
Unoptimized (Onone)Regression (12)
Improvement (10)
No Changes (312)
Hardware Overview
|
public func _customContainsEquatableElement( | ||
_ element: Element | ||
) -> Bool? { | ||
if element < _start || _end <= element { |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Is it worth adding an overload for when Element: FixedWidthInteger
that would compute contains entirely?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Hm...
@swift-ci please smoke test compiler performance |
!!! Couldn't read commit file !!! |
No description provided.