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

C++20 #456

Open
wants to merge 6 commits into
base: master
Choose a base branch
from
Open

C++20 #456

wants to merge 6 commits into from

Conversation

Matthies
Copy link
Owner

Non-regression test:
Elo | 1.99 +- 3.56 (95%)
SPRT | 10.0+0.10s Threads=1 Hash=8MB
LLR | 2.96 (-2.94, 2.94) [-3.00, 1.00]
Games | N: 17838 W: 4402 L: 4300 D: 9136
Penta | [53, 1922, 4879, 2000, 65]

@Matthies Matthies self-assigned this Mar 11, 2024
@Matthies
Copy link
Owner Author

Matthies commented Mar 13, 2024

There are several (almost all) workers at OpenBench using gcc 8 or 9 which don't understand -std=c++20 and suggest using -std=c++2a. Not working in OB is a 'no go' merging this branch but I will give c++2a a try.

@Matthies
Copy link
Owner Author

Matthies commented Jun 3, 2024

Switching to c++2a for workers with older gcc9 works technically but the performance of the code of this old compiler seems very bad looking at a fixed 1000 games test:

Elo   | -16.66 +- 10.95 (95%)
Conf  | 10.0+0.10s Threads=1 Hash=8MB
Games | N: 1002 W: 231 L: 279 D: 492
Penta | [11, 121, 275, 93, 1]
http://chess.grantnet.us/test/35899/

So I will stay at C++11 for now.

@Matthies
Copy link
Owner Author

Matthies commented Aug 2, 2024

If I understand correctly. You want to switch from C++11 to C++20. It is better to switch to C++17 for now. Many users still use Visual Studio 2019, which does not support C++20.

You don't understand correctly and I don't care, which VS version people use.

@Matthies
Copy link
Owner Author

Matthies commented Aug 2, 2024

... it could be that we end up writing ...

You will end up writing here.

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.

1 participant