Skip to content

ArrayVTables: ArrayCommon#6838

Open
gatesn wants to merge 1 commit intodevelopfrom
ngates/array-vtable-common
Open

ArrayVTables: ArrayCommon#6838
gatesn wants to merge 1 commit intodevelopfrom
ngates/array-vtable-common

Conversation

@gatesn
Copy link
Contributor

@gatesn gatesn commented Mar 7, 2026

Introduces an ArrayCommon struct that all arrays hold.

We will later invert this to be Array<V> with the common fields hoisted above the other array data.

Signed-off-by: Nicholas Gates <nick@nickgates.com>
@gatesn gatesn added the changelog/chore A trivial change label Mar 7, 2026
Copy link
Contributor Author

gatesn commented Mar 7, 2026

This stack of pull requests is managed by Graphite. Learn more about stacking.

@codspeed-hq
Copy link

codspeed-hq bot commented Mar 7, 2026

Merging this PR will improve performance by 26.42%

⚡ 5 improved benchmarks
✅ 995 untouched benchmarks
⏩ 1466 skipped benchmarks1

Performance Changes

Mode Benchmark BASE HEAD Efficiency
Simulation patched_take_200k_first_chunk_only 5.4 ms 4.8 ms +11.97%
Simulation take_200k_dispersed 4.5 ms 3.6 ms +24.34%
Simulation patched_take_200k_dispersed 5.6 ms 4.7 ms +19.84%
Simulation take_200k_first_chunk_only 4.2 ms 3.3 ms +26.42%
Simulation rebuild_naive 267.1 µs 216.9 µs +23.16%

Comparing ngates/array-vtable-common (c5bdf94) with develop (5ec8d2f)

Open in CodSpeed

Footnotes

  1. 1466 benchmarks were skipped, so the baseline results were used instead. If they were deleted from the codebase, click here and archive them to remove them from the performance reports.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

changelog/chore A trivial change

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant