-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add proper build system which automatically detects architecture (or at least add aarch64 support) #2355
Comments
Adding an aarch64 option in the makefile: sure, we could consider this. However, the experience of aarch64 in the SF team is quite limited, so maybe you would be the best person to propose a patch as you obviously have access to such a machine? About cmake or meson: probably not an option, because each build tool we add needs extra maintenance, and we prefer to keep SF toolchain as simple as possible. |
Please add |
I assume you have access ? In that case, suggest patches (as a pull request or as a diff), indicating how they have been tested. |
There already is cmake/Visual Studio support but it's autogenerated by appveyor.yml I extracted it and shortened it here: https://github.com/bftjoe/Stockfish/blob/master/CMakeLists.txt Not sure how it's more maintenance to include CMakeLists.txt in the repo instead of autogenerating it... |
try its options prefixed by CXXFLAGS= in place of ARC= COMP=. -mabi=name With the latter choice the options ‘-mstack-protector-guard-reg=reg’ and -mtrack-speculation -march=name -mtune=name -mcpu=name |
Tested with bench run after compiling with - g++ (Debian 6.3.0-18+deb9u1) 6.3.0 20170516 - clang version 3.8.1-24 on ThunderX CN8890. No functional change.
Here's a minimal patch to maybe make some progress on this: #2760 |
Thanks, must God bless you.. ameen |
I'll merge the pull request to support armv8 and close the issue with it. |
It's a little bit messy to build stockfish for multiple architectures by passing the
ARCH
variable. Is adding a build system like cmake or meson something that you would consider?Also there is currently not an architecture option for
aarch64
and it's not clear from the makefile how I can manually override this easily. So, I when I included stockfish as a module forgnome-chess
at flathub I chose to usearmv7
even for theaarch64
architecture as you can see here:flathub/org.gnome.Chess@01ff82e
Maybe you can add an
aarch64
option too at the makefile, until you consider the decision on the build system?The text was updated successfully, but these errors were encountered: