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

Enhancement: enable SIMD convolution by default #1213

Closed
lovell opened this issue Apr 29, 2018 · 1 comment
Closed

Enhancement: enable SIMD convolution by default #1213

lovell opened this issue Apr 29, 2018 · 1 comment

Comments

@lovell
Copy link
Owner

lovell commented Apr 29, 2018

Many operations in libvips already use SIMD via compile-time auto-vectorisation. The simd() switch enables additional run-time SIMD for operations involving convolution such as blur, sharpen and resize.

http://sharp.pixelplumbing.com/en/stable/api-utility/#simd

This is currently an optional setting that defaults to false.

This proposal is to default the setting to true and therefore instruct libvips to take further advantage of SIMD instructions via liborc, already included in the prebuilt binaries.

The current version of liborc is v0.4.28, which has proven itself to be stable in systems that resize millions of images per day.

Does anyone, especially those who already enable this feature under heavy load, have any reservations about making this change?

@lovell lovell added this to the v0.21.0 milestone Apr 29, 2018
lovell added a commit that referenced this issue Sep 19, 2018
@lovell lovell changed the title Proposal: enable SIMD for convolution by default Enchancement: enable SIMD convolution by default Sep 19, 2018
@lovell
Copy link
Owner Author

lovell commented Oct 4, 2018

v0.21.0 now available with this change.

@lovell lovell closed this as completed Oct 4, 2018
@lovell lovell changed the title Enchancement: enable SIMD convolution by default Enhancement: enable SIMD convolution by default Nov 27, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

1 participant