Skip to content

Test SSSE3 percent encode cutoff - #6

Open
thisalihassan wants to merge 9 commits into
mainfrom
codspeed-percent-encode-ssse3-cutoff
Open

Test SSSE3 percent encode cutoff#6
thisalihassan wants to merge 9 commits into
mainfrom
codspeed-percent-encode-ssse3-cutoff

Conversation

@thisalihassan

@thisalihassan thisalihassan commented May 30, 2026

Copy link
Copy Markdown
Owner

Fork-only CodSpeed experiment for \n\nThis keeps the SSSE3 percent-encode path for longer inputs, but uses the scalar path for short inputs to avoid LUT setup overhead in short percent_encode benchmarks.

Signed-off-by: Ali Hassan <ali-hassan27@outlook.com>
Signed-off-by: Ali Hassan <ali-hassan27@outlook.com>
…de-ssse3-cutoff

# Conflicts:
#	benchmarks/percent_encode.cpp
@codspeed-hq

codspeed-hq Bot commented May 30, 2026

Copy link
Copy Markdown

Merging this PR will improve performance by 12.84%

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

⚡ 1 improved benchmark
❌ 1 regressed benchmark
✅ 30 untouched benchmarks

Warning

Please fix the performance issues or acknowledge them on CodSpeed.

Performance Changes

Benchmark BASE HEAD Efficiency
DecodeDense 6.7 µs 7.5 µs -11.04%
url_search_params_AdaURL 196.1 µs 137 µs +43.13%

Tip

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


Comparing codspeed-percent-encode-ssse3-cutoff (370f532) with main (08cd431)

Open in CodSpeed

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.

1 participant