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

avoid SVE/SVE2 on old compilers. Thanks @Xarbirus, Refs #1881 #2103

Merged
merged 1 commit into from
Apr 22, 2024

Conversation

copybara-service[bot]
Copy link
Contributor

avoid SVE/SVE2 on old compilers. Thanks @Xarbirus, Refs #1881

@copybara-service copybara-service bot force-pushed the test_626969398 branch 2 times, most recently from 8249da8 to 7db71ec Compare April 22, 2024 11:09
@copybara-service copybara-service bot merged commit d4f7911 into master Apr 22, 2024
@copybara-service copybara-service bot deleted the test_626969398 branch April 22, 2024 11:11
@Mousius
Copy link
Contributor

Mousius commented May 6, 2024

@jan-wassenberg I saw the issue refers to SVE2, but not SVE, is there a reason to disable both? I think this is causing the detect_targets.h error in numpy/numpy#25781 as there's no longer an attainable SVE target for GCC 11.

@jan-wassenberg
Copy link
Member

Thanks for pointing that out. I had disabled SVE as well based on gcc not being able to compile, but this was because I still had armv9- in the arch string. Looks like GCC 8 is sufficient.
The detect_targets issue also indicates a mismatch between disabling here, and enabling for the baseline. I will also harmonize the two.

copybara-service bot pushed a commit that referenced this pull request May 7, 2024
copybara-service bot pushed a commit that referenced this pull request May 7, 2024
copybara-service bot pushed a commit that referenced this pull request May 7, 2024
copybara-service bot pushed a commit that referenced this pull request May 7, 2024
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.

2 participants