Skip to content

Perf/packed api - #21

Merged
jcfangc merged 9 commits into
mainfrom
perf/packed-api
Sep 14, 2026
Merged

jcfangc merged 9 commits into
mainfrom
perf/packed-api

Conversation

@jcfangc

@jcfangc jcfangc commented Sep 14, 2026

Copy link
Copy Markdown
Owner

No description provided.

@codecov-commenter

Copy link
Copy Markdown

⚠️ Please install the 'codecov app svg image' to ensure uploads and comments are reliably processed by Codecov.

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@codspeed

codspeed Bot commented Sep 14, 2026

Copy link
Copy Markdown

Merging this PR will regress 1 benchmark

⚠️ Different runtime environments detected

Some benchmarks with significant performance changes were compared across different runtime environments,
which may affect the accuracy of the results.

Open the report in CodSpeed to investigate

⚡ 13 improved benchmarks
❌ 1 regressed benchmark
✅ 213 untouched benchmarks
🆕 516 new benchmarks
⏩ 210 skipped benchmarks1

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ cmp/len_64/identical/ours_string 499.7 ns 558.1 ns -10.45%
⚡ count_ones/len_4096/all_zeros/leading/ours_string 180.6 ns 151.4 ns +19.27%
⚡ count_ones/len_65/all_zeros/leading/ours_string 180.6 ns 151.4 ns +19.27%
⚡ count_zeros/len_65/dense/leading/ours_str 183.9 ns 154.7 ns +18.85%
⚡ count_zeros/len_65/dense/trailing/ours_str 250.3 ns 221.1 ns +13.19%
⚡ count_zeros/len_65/alternating/trailing/ours_str 251.4 ns 222.2 ns +13.13%
⚡ count_ones/len_4096/all_zeros/leading/ours_str 271.7 ns 242.5 ns +12.03%
⚡ count_ones/len_65/all_zeros/leading/ours_str 271.7 ns 242.5 ns +12.03%
⚡ count_ones/len_65/all_zeros/trailing/ours_str 276.4 ns 247.2 ns +11.8%
⚡ count_ones/len_65/dense/trailing/ours_str 277.8 ns 248.6 ns +11.73%
⚡ count_ones/len_65/alternating/ours_string 278.6 ns 249.4 ns +11.69%
⚡ count_ones/len_65/dense/ours_string 278.6 ns 249.4 ns +11.69%
⚡ count_ones/len_65/sparse/ours_string 278.6 ns 249.4 ns +11.69%
⚡ count_zeros/len_65/all_zeros/trailing/ours_str 279.7 ns 250.6 ns +11.64%
🆕 packed_access/get_random_stream/bits_1_len_1048576/ours_packed_str_unaligned N/A 17.9 ms N/A
🆕 packed_access/get_random_stream/bits_1_len_1048576/ours_packed_string N/A 14.2 ms N/A
🆕 packed_access/get_sequential/bits_1_len_1048576/ours_packed_str_unaligned N/A 11.4 ms N/A
🆕 packed_access/get_sequential/bits_1_len_1048576/ours_packed_string N/A 9.1 ms N/A
🆕 packed_access/get_random_stream/bits_1_len_64/ours_packed_str_unaligned N/A 1.4 µs N/A
🆕 packed_access/get_random_stream/bits_1_len_64/ours_packed_string N/A 994.7 ns N/A
... ... ... ... ...

ℹ️ Only the first 20 benchmarks are displayed. Go to the app to view all benchmarks.

Tip

Investigate this regression by commenting @codspeedbot fix this regression on this PR, or directly use the CodSpeed MCP with your agent.


Comparing perf/packed-api (3a73633) with main (4e3b6b7)

Open in CodSpeed

Footnotes

  1. 210 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. ↩

@jcfangc
jcfangc merged commit 2e89b8e into main Sep 14, 2026
6 of 7 checks passed
@jcfangc
jcfangc deleted the perf/packed-api branch September 14, 2026 13:18
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.

2 participants