-
-
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
Add AVX2 version of GetResidualCost and SSE2 version of SetCoeffs #1902
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1902 +/- ##
======================================
Coverage 87% 87%
======================================
Files 961 961
Lines 50990 51034 +44
Branches 6319 6324 +5
======================================
+ Hits 44747 44793 +46
Misses 5202 5202
+ Partials 1041 1039 -2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
|
||
#if SUPPORTS_RUNTIME_INTRINSICS | ||
[Fact] | ||
public void RunEncodeLossy_WithPeakImage_WithHardwareIntrinsics_Works() => FeatureTestRunner.RunWithHwIntrinsicsFeature(RunEncodeLossy_WithPeakImage, HwIntrinsics.AllowAll); |
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.
Note: I could not figure out how to write a single test for GetResidualCost
, it would be rather complicated. Instead i have decided to do lossy encoding test with and without hardware intrinsics
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 PR adds a AVX2 version of
GetResidualCost
and SSE2 version ofSetCoeffs
, which is used during webp lossy encoding.Related to #1786
Profiling results:
master
PR
The performance gain from
GetResidualCost
is a bit underwhelming, but its not nothing either.