-
Notifications
You must be signed in to change notification settings - Fork 10.5k
Concrete SIMD operations, part 1 #36172
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 compiler performance |
Compilation-performance test failed |
@swift-ci please smoke test compiler performance |
bc67cb2
to
d845a1e
Compare
@swift-ci please smoke test compiler performance |
Summary for main smoketestRegressions found (see below) Debugdebug briefNone debug detailedNone Releaserelease briefNone release detailedNone |
I think there is a network outage happening in CI right now. /cc @shahmishal |
@swift-ci please smoke test compiler performance |
!!! Couldn't read commit file !!! |
@swift-ci please smoke test compiler performance |
Compilation-performance test failed |
@swift-ci please smoke test compiler performance |
!!! Couldn't read commit file !!! |
Compilation-performance test failed |
2 similar comments
Compilation-performance test failed |
Compilation-performance test failed |
Summary for main smoketestRegressions found (see below) Debugdebug briefRegressed (1)
Improved (1)
Unchanged (delta < 1.0% or delta < 100.0ms) (1)
debug detailedRegressed (46)
Improved (4)
Unchanged (delta < 1.0% or delta < 100.0ms) (214)
Releaserelease briefRegressed (0)
Improved (0)
Unchanged (delta < 1.0% or delta < 100.0ms) (3)
release detailedRegressed (0)
Improved (4)
Unchanged (delta < 1.0% or delta < 100.0ms) (260)
|
@swift-ci please benchmark |
@swift-ci please test source compatibility |
Performance: -O
Code size: -OPerformance: -Osize
Code size: -OsizePerformance: -Onone
Code size: -swiftlibs
How to read the dataThe tables contain differences in performance which are larger than 8% and differences in code size which are larger than 1%.If you see any unexpected regressions, you should consider fixing the Noise: Sometimes the performance results (not code size!) contain false Hardware Overview
|
+,-,*,/ in particular are problematic because the differentiation module wants to make the differentiable, but @_aEIC functions cannot be differentiable. So we'll simply back them out for now.
9851268
to
0343180
Compare
@swift-ci benchmark |
@swift-ci test compiler performance |
@swift-ci test |
Performance: -O
Code size: -O
Performance: -Osize
Code size: -Osize
Performance: -Onone
Code size: -swiftlibsHow to read the dataThe tables contain differences in performance which are larger than 8% and differences in code size which are larger than 1%.If you see any unexpected regressions, you should consider fixing the Noise: Sometimes the performance results (not code size!) contain false Hardware Overview
|
Build failed |
@swift-ci please smoke test macos |
@swift-ci test compiler performance |
Summary for main fullUnexpected test results, excluded stats for RxCocoa Regressions found (see below) Debug-batchdebug-batch briefRegressed (0)
Improved (0)
Unchanged (delta < 1.0% or delta < 100.0ms) (3)
debug-batch detailedRegressed (13)
Improved (0)
Unchanged (delta < 1.0% or delta < 100.0ms) (251)
Releaserelease briefRegressed (0)
Improved (0)
Unchanged (delta < 1.0% or delta < 100.0ms) (3)
release detailedRegressed (1)
Improved (0)
Unchanged (delta < 1.0% or delta < 100.0ms) (263)
|
Adds concrete overloads of the following SIMD operations:
This makes some simple benchmarks 10-100x faster, which is basically a no-brainer, while staying away from the most heavily used operators, so hopefully doesn't impact compilation performance too badly.