Skip to content

reduce Hit[][][] memory allocations with a pool - #1515

Open
baltzell wants to merge 4 commits into
developmentfrom
cvt-alloc
Open

baltzell wants to merge 4 commits into
developmentfrom
cvt-alloc

Conversation

@baltzell

@baltzell baltzell commented Oct 14, 2026 •

Copy link
Copy Markdown
Collaborator

It's a few-% of our memory allocations, according to asprof. And looks like about 50-million, 32-bit allocations per call (2 per event), based on tallying up the array dimensions and Hit size.

Note, the 64-size pool is good enough unless this were to start getting used by multiple engines.

@baltzell
baltzell marked this pull request as ready for review October 14, 2026 21:13
@baltzell baltzell changed the title reduce memory allocations with a pool reduce Hit[][][] memory allocations with a pool Oct 14, 2026
@baltzell baltzell added the speed label Oct 15, 2026
tongtongcao
tongtongcao previously approved these changes Oct 15, 2026

@tongtongcao tongtongcao 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.

Results are identical before and after updates.

This branch has not been deployed

No deployments
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.

2 participants