Skip to content

Avoid copying the accumulator twice - #356

Merged
kevlu8 merged 1 commit into
mainfrom
dont-copy-acc-twice
Aug 7, 2026
Merged

Avoid copying the accumulator twice#356
kevlu8 merged 1 commit into
mainfrom
dont-copy-acc-twice

Conversation

@kevlu8

@kevlu8 kevlu8 commented Aug 7, 2026

Copy link
Copy Markdown
Owner 
Elo   | 2.58 +- 3.15 (95%)
SPRT  | 4.0+0.04s Threads=1 Hash=32MB
LLR   | 3.04 (-2.25, 2.89) [-4.00, 0.00]
Games | N: 13464 W: 3449 L: 3349 D: 6666
Penta | [105, 1462, 3499, 1560, 106]

https://ob.int0x80.ca/test/982/

Bench: 2186656

@kevlu8
kevlu8 requested a review from wdotmathree August 7, 2026 19:54
wdotmathree
wdotmathree approved these changes Aug 7, 2026
View reviewed changes

@wdotmathree wdotmathree left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@kevlu8
kevlu8 merged commit 3402aa3 into main Aug 7, 2026
9 checks passed
@kevlu8
kevlu8 deleted the dont-copy-acc-twice branch August 7, 2026 21:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@wdotmathree wdotmathree wdotmathree approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@kevlu8 @wdotmathree