-
-
Notifications
You must be signed in to change notification settings - Fork 851
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
SSE41 version of Hadamard transform #1810
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1810 +/- ##
==========================================
- Coverage 87.31% 87.12% -0.20%
==========================================
Files 936 936
Lines 48015 48085 +70
Branches 6034 6035 +1
==========================================
- Hits 41924 41892 -32
- Misses 5092 5190 +98
- Partials 999 1003 +4
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
…msse # Conflicts: # src/ImageSharp/Formats/Webp/Lossy/LossyUtils.cs
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.
LGTM.
Prerequisites
Description
This adds a SSE41 version of the Hadamard transform which is used during lossy webp encoding.
Relates to #1786
Profiling results:
Before:
After:
note the number of calls is halfed, because the sse version does two transformas at once