Skip to content

Feat/packed string - #19

Merged
jcfangc merged 79 commits into
mainfrom
feat/packed-string
Sep 5, 2026
Merged

jcfangc merged 79 commits into
mainfrom
feat/packed-string

Conversation

@jcfangc

@jcfangc jcfangc commented Sep 5, 2026

Copy link
Copy Markdown
Owner

No description provided.

@codspeed

codspeed Bot commented Sep 5, 2026 •

Copy link
Copy Markdown

Merging this PR will regress 10 benchmarks

⚠️ 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

⚡ 17 improved benchmarks
❌ 10 regressed benchmarks
✅ 200 untouched benchmarks
⏩ 210 skipped benchmarks1

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
❌ ends_with/len_65536/miss/ours_str_string 442.8 ns 560 ns -20.93%
❌ ends_with/len_65536/miss/ours_string_str 442.8 ns 559.4 ns -20.85%
❌ ends_with/len_65536/hit/ours_str_string 472.8 ns 590 ns -19.87%
❌ ends_with/len_65536/hit/ours_string_str 472.8 ns 589.4 ns -19.79%
❌ ends_with/len_65536/miss/ours_str_str 443.3 ns 531.4 ns -16.57%
❌ ends_with/len_65536/miss/ours_string_string 442.2 ns 529.7 ns -16.52%
❌ ends_with/len_65536/hit/ours_str_str 473.3 ns 561.4 ns -15.69%
❌ ends_with/len_65536/hit/ours_string_string 472.2 ns 559.7 ns -15.63%
❌ cmp/len_65/diff_last/ours_string 596.4 ns 683.9 ns -12.79%
❌ 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%
⚡ matches_at_str/len_65536/no/aligned/ours_string_str 548.3 ns 460.8 ns +18.99%
⚡ count_zeros/len_65/dense/leading/ours_str 183.9 ns 154.7 ns +18.85%
⚡ matches_at_str/len_65536/yes/aligned/ours_string_str 611.1 ns 523.6 ns +16.71%
⚡ matches_at_str/len_65/no/unaligned/ours_string_str 490.3 ns 431.1 ns +13.72%
⚡ matches_at_str/len_65/yes/unaligned/ours_string_str 490.3 ns 431.1 ns +13.72%
⚡ 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%
... ... ... ... ...

ℹ️ 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 feat/packed-string (27ba5cb) with main (fb56862)

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 241871a into main Sep 5, 2026
6 of 7 checks passed
@jcfangc
jcfangc deleted the feat/packed-string branch September 5, 2026 06:04
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