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

Optimize update_accumulator_refresh_cache() #5239

Closed
wants to merge 1 commit into from

Conversation

mstembera
Copy link
Contributor

STC https://tests.stockfishchess.org/tests/view/664105df26ac5f9b286d30e6
LLR: 2.94 (-2.94,2.94) <0.00,2.00>
Total: 178528 W: 46235 L: 45750 D: 86543
Ptnml(0-2): 505, 17792, 52142, 18363, 462

Combo of two yellow speedups
https://tests.stockfishchess.org/tests/view/6640abf9d163897c63214f5c
LLR: -2.93 (-2.94,2.94) <0.00,2.00>
Total: 355744 W: 91714 L: 91470 D: 172560
Ptnml(0-2): 913, 36233, 103384, 36381, 961
https://tests.stockfishchess.org/tests/view/6628ce073fe04ce4cefc739c
LLR: -2.93 (-2.94,2.94) <0.00,2.00>
Total: 627040 W: 162001 L: 161339 D: 303700
Ptnml(0-2): 2268, 72379, 163532, 73105, 2236

No functional change
bench: 2353057

No functional change
bench: 2353057
Copy link

clang-format 17 needs to be run on this PR.
If you do not have clang-format installed, the maintainer will run it when merging.
For the exact version please see https://packages.ubuntu.com/mantic/clang-format-17.

(execution 9053904550 / attempt 1)

@Torom
Copy link
Contributor

Torom commented May 12, 2024

ARCH=x86-64-bmi2 COMP=clang

Result of  50 runs
==================
base (...es/stockfish) =     956345  +/- 1452
test (...ockfish.ACRT) =     959198  +/- 1631
diff                   =      +2853  +/- 2332

speedup        = +0.0030
P(speedup > 0) =  0.9917

CPU: 4 x Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz
Hyperthreading: on

ARCH=x86-64-bmi2 COMP=gcc

Result of  50 runs
==================
base (...tockfish_gcc) =     953245  +/- 1214
test (...ish.ACRT_gcc) =     965634  +/- 1733
diff                   =     +12388  +/- 2482

speedup        = +0.0130
P(speedup > 0) =  1.0000

CPU: 4 x Intel(R) Core(TM) i7-6700K CPU @ 4.00GHz
Hyperthreading: on

@vondele vondele added 🚀 gainer Gains elo no-functional-change to be merged Will be merged shortly labels May 13, 2024
@vondele vondele closed this in e608eab May 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants