Better arm support #139
Merged
Better arm support #139
Conversation
This file contains 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
The main contribution of this PR is to improve the build process.
This was instigated by changes in GitHub runners, whereby Intel-based Mac machines are not longer employed.
We modify two options from CMakeLists.txt, which are
We also added a few options to disable using particular instruction sets: SSE, SSE2, SSSE3, SSE4, AVX, AVX2 and AVX512. It is left to the end user to decide what best suits them. Removing an option can be due to that it is missing from the compiler, or to reduce the resulting code size.
Notice that all instructions are enabled by default and need to be disabled if desired,
We add better support for ARM -- No NEON code yet.
Also added more Actions.
This should address issues #111 and #121.