Skip to content
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

Unify register assignment in NEON assembly for get_sad #2607

Merged
merged 1 commit into from
Dec 4, 2023

Conversation

barrbrain
Copy link
Collaborator

Easier to read and reduces compressed compiled object size by 10%.

@coveralls
Copy link
Collaborator

Coverage Status

Coverage increased (+2.8%) to 68.708% when pulling b5cad8a on barrbrain:aarch64-sad into b6a488f on xiph:master.

@shssoichiro
Copy link
Collaborator

It seems like something isn't quite happy here: https://travis-ci.org/github/xiph/rav1e/builds/748500930

@barrbrain barrbrain marked this pull request as ready for review November 30, 2023 05:30
@barrbrain barrbrain requested a review from lu-zero November 30, 2023 05:33
Copy link

codecov bot commented Nov 30, 2023

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (4a33359) 88.22% compared to head (60dd8a8) 88.22%.
Report is 1 commits behind head on master.

Additional details and impacted files
@@           Coverage Diff           @@
##           master    #2607   +/-   ##
=======================================
  Coverage   88.22%   88.22%           
=======================================
  Files          87       87           
  Lines       28221    28221           
=======================================
  Hits        24898    24898           
  Misses       3323     3323           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@barrbrain barrbrain force-pushed the aarch64-sad branch 3 times, most recently from bf3d919 to bd233ee Compare December 1, 2023 16:38
Easier to read and reduces compressed compiled object size by 10%.
Copy link
Collaborator

@lu-zero lu-zero left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

do we have other places that would enjoy ldp?

@barrbrain barrbrain merged commit ceb1909 into xiph:master Dec 4, 2023
25 checks passed
@barrbrain
Copy link
Collaborator Author

do we have other places that would enjoy ldp?

For sure, most of the wider distortion kernels. I will revisit them when we have a complete set of distortion assembly.

@barrbrain barrbrain deleted the aarch64-sad branch December 4, 2023 14:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants