-
Notifications
You must be signed in to change notification settings - Fork 674
Add Google Highway SIMD acceleration for ImageBufAlgo operations #4986
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
Closed
Closed
Changes from 19 commits
Commits
Show all changes
21 commits
Select commit
Hold shift + click to select a range
c8d2dbd
Add Google Highway SIMD resampling to ImageBufAlgo
ssh4net f13876b
Add Highway-based SIMD fast paths for pixel math ops
ssh4net 0ea3dff
Remove all GitHub Actions workflow files
ssh4net b1b8be0
Refactor SIMD math to use imagebufalgo_hwy_pvt.h
ssh4net 2cac746
Improve SIMD support and type handling in imagebufalgo
ssh4net 2c0f517
Add HWY arithmetic and resample benchmark scripts
ssh4net 9c5808b
Add SIMD-optimized range and premult/unpremult pixel math
ssh4net 20960da
Add Highway SIMD toggle and update pixel math logic
ssh4net f0552f7
Revamp Highway SIMD benchmark with detailed tests
ssh4net 7229160
Apply pixel-center convention in resample_hwy interpolation
ssh4net ed0c40b
Move and integrate hwy_test into src directory
ssh4net d6809ff
Add SIMD (Highway) acceleration for pixel math ops
ssh4net d6fbbc7
Add clamp benchmark and format code in mad_impl_hwy
ssh4net 45d6beb
Optimize ImageBufAlgo integer ops with native SIMD
ssh4net b87f054
Add Highway SIMD for invert and contrast_remap ops
ssh4net 1640275
Normalize integer image data to 0-1 in SIMD math
ssh4net 53b0294
Revert "Remove all GitHub Actions workflow files"
ssh4net f2fe675
Improve SIMD normalization for signed integer image types
ssh4net 699196b
Update hwy_test.cpp
ssh4net 87421c4
Update imagebufalgo_xform.cpp
ssh4net 37089ae
Refactor ImageBufAlgo SIMD helpers and usage
ssh4net File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,5 @@ | ||
| # Copyright Contributors to the OpenImageIO project. | ||
| # SPDX-License-Identifier: Apache-2.0 | ||
| # https://github.com/AcademySoftwareFoundation/OpenImageIO | ||
|
|
||
| fancy_add_executable (LINK_LIBRARIES OpenImageIO) |
Oops, something went wrong.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
I would prefer this be an optional dependency,
And then
#ifdef USE_HWYin the cpp files to conditionally compile the parts that need hwy.If we add a hard dependency, it precludes backporting any part of it to 3.1, since we don't change minimum dependencies in a released branch.