Skip to content

Use SWAR in SmileParser - #757

Merged
cowtowncoder merged 4 commits into
FasterXML:3.xfrom
pjfanning:smile-parser
Aug 22, 2026
Merged

Use SWAR in SmileParser#757
cowtowncoder merged 4 commits into
FasterXML:3.xfrom
pjfanning:smile-parser

Conversation

@pjfanning

@pjfanning pjfanning commented Aug 21, 2026

Copy link
Copy Markdown
Member

Similar change to #756 and #752

I added a benchmark to https://github.com/pjfanning/jackson-bench

Benchmark                                                  Mode  Cnt     Score    Error  Units
SmileDoubleArrayBench.benchParseFromByteArray             thrpt    5   455.498 ± 32.719  ops/s
SmileDoubleArrayBench.benchParseFromByteArraySwar         thrpt    5   473.796 ± 21.485  ops/s
SmileNameDecodeBench.benchParseFromByteArray              thrpt    5  1227.329 ± 29.590  ops/s
SmileNameDecodeBench.benchParseFromByteArraySwar          thrpt    5  1327.239 ± 28.582  ops/s

@pjfanning
pjfanning marked this pull request as draft August 21, 2026 23:45
@github-actions

github-actions Bot commented Aug 21, 2026

Copy link
Copy Markdown

🧪 Code Coverage Report

Coverage Type Coverage Change
📝 Instructions 76.71% 📉 -0.24%
🔀 Branches 68.28% 📉 -0.19%

@pjfanning pjfanning changed the title WIP: use SWAR in SmileParser use SWAR in SmileParser Aug 22, 2026
@pjfanning
pjfanning marked this pull request as ready for review August 22, 2026 01:08
@cowtowncoder cowtowncoder changed the title use SWAR in SmileParser Use SWAR in SmileParser Aug 22, 2026
@cowtowncoder cowtowncoder changed the title Use SWAR in SmileParser Use SWAR in SmileParser Aug 22, 2026
@cowtowncoder cowtowncoder added the performance Issue/PR related to performance; usually optimization there-of label Aug 22, 2026
@cowtowncoder
cowtowncoder merged commit 7f7e0c8 into FasterXML:3.x Aug 22, 2026
3 checks passed
@pjfanning
pjfanning deleted the smile-parser branch August 22, 2026 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

performance Issue/PR related to performance; usually optimization there-of smile

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants