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

build: Fix broken OIIO_NO_NEON definition #3911

Merged
merged 1 commit into from
Jul 17, 2023

Conversation

lgritz
Copy link
Collaborator

@lgritz lgritz commented Jul 14, 2023

Clean up the #if tests that disable Intel SIMD when not on Intel architectures, which incorrectly disabled ARM NEON when on ARM.

Fixes #3909

Clean up the `#if` tests that disable Intel SIMD when not on Intel
architectures, which incorrectly disabled ARM NEON when on ARM.

Fixes 3909

Signed-off-by: Larry Gritz <[email protected]>
@lgritz
Copy link
Collaborator Author

lgritz commented Jul 17, 2023

@1-Olivier reports that this fixes the reported problem

@lgritz lgritz merged commit 780fe68 into AcademySoftwareFoundation:master Jul 17, 2023
22 of 23 checks passed
lgritz added a commit to lgritz/OpenImageIO that referenced this pull request Jul 17, 2023
…3911)

Clean up the `#if` tests that disable Intel SIMD when not on Intel
architectures, which incorrectly disabled ARM NEON when on ARM.

Fixes AcademySoftwareFoundation#3909

Signed-off-by: Larry Gritz <[email protected]>
@lgritz lgritz deleted the lg-neon2 branch July 30, 2023 23:48
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.

[BUG] neon support is disabled on macOS arm
1 participant