Skip to content

Conversation

@dduan
Copy link
Owner

@dduan dduan commented Dec 30, 2025

Now that there's only one implementation, it can live in Parser.swift

Now that there's only one implementation, it can live in Parser.swift
@dduan dduan enabled auto-merge (squash) December 30, 2025 18:39
@github-actions
Copy link

Comparing results between 'main' and 'pull_request'

Host 'runnervmh13bl' with 4 'x86_64' processors with 15 GB memory, running:
#18~24.04.1-Ubuntu SMP Sat Jun 28 04:46:03 UTC 2025

TOMLDecoderBenchmarks

Decode toml.io example metrics

Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (μs) * p0 p25 p50 p75 p90 p99 p100 Samples
main 70 71 72 72 73 81 123 8715
pull_request 70 72 72 73 74 82 119 8696
Δ 0 1 0 1 1 1 -4 -19
Improvement % 0 -1 0 -1 -1 -1 3 -19

Retains: results within specified thresholds, fold down for details.

Retains * p0 p25 p50 p75 p90 p99 p100 Samples
main 625 625 625 625 625 625 625 8715
pull_request 624 624 624 624 624 624 624 8696
Δ -1 -1 -1 -1 -1 -1 -1 -19
Improvement % 0 0 0 0 0 0 0 -19

Parse toml.io example metrics

Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (ns) * p0 p25 p50 p75 p90 p99 p100 Samples
main 5824 6007 6063 6135 6223 7191 21976 10000
pull_request 5678 5899 5959 6027 6123 6991 19487 10000
Δ -146 -108 -104 -108 -100 -200 -2489 0
Improvement % 3 2 2 2 2 3 11 0

Retains: results within specified thresholds, fold down for details.

Retains * p0 p25 p50 p75 p90 p99 p100 Samples
main 8 9 9 9 9 9 9 10000
pull_request 8 9 9 9 9 9 9 10000
Δ 0 0 0 0 0 0 0 0
Improvement % 0 0 0 0 0 0 0 0

decode canada.toml metrics

Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (ms) * p0 p25 p50 p75 p90 p99 p100 Samples
main 276 277 277 278 278 278 278 4
pull_request 266 266 267 268 275 275 275 4
Δ -10 -11 -10 -10 -3 -3 -3 0
Improvement % 4 4 4 4 1 1 1 0

Retains: results within specified thresholds, fold down for details.

Retains (K) * p0 p25 p50 p75 p90 p99 p100 Samples
main 1674 1674 1674 1674 1674 1674 1674 4
pull_request 1674 1674 1674 1674 1674 1674 1674 4
Δ 0 0 0 0 0 0 0 0
Improvement % 0 0 0 0 0 0 0 0

decode twitter.toml metrics

Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (μs) * p0 p25 p50 p75 p90 p99 p100 Samples
main 9167 9232 9257 9273 9322 9716 9978 108
pull_request 9110 9167 9191 9224 9249 9421 9662 108
Δ -57 -65 -66 -49 -73 -295 -316 0
Improvement % 1 1 1 1 1 3 3 0

Retains: results within specified thresholds, fold down for details.

Retains (K) * p0 p25 p50 p75 p90 p99 p100 Samples
main 66 66 66 66 66 66 66 108
pull_request 66 66 66 66 66 66 66 108
Δ 0 0 0 0 0 0 0 0
Improvement % 0 0 0 0 0 0 0 0

parse canada.toml metrics

Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (ms) * p0 p25 p50 p75 p90 p99 p100 Samples
main 17 18 19 19 20 21 21 54
pull_request 18 19 19 20 20 21 21 53
Δ 1 1 0 1 0 0 0 -1
Improvement % -6 -6 0 -5 0 0 0 -1

Retains: results within specified thresholds, fold down for details.

Retains * p0 p25 p50 p75 p90 p99 p100 Samples
main 132 133 133 133 133 133 133 54
pull_request 132 133 133 133 133 133 133 53
Δ 0 0 0 0 0 0 0 -1
Improvement % 0 0 0 0 0 0 0 -1

parse twitter.toml metrics

Time (wall clock): results within specified thresholds, fold down for details.

Time (wall clock) (μs) * p0 p25 p50 p75 p90 p99 p100 Samples
main 2754 2771 2777 2785 2798 2853 3194 355
pull_request 2700 2718 2726 2734 2744 2968 3160 361
Δ -54 -53 -51 -51 -54 115 -34 6
Improvement % 2 2 2 2 2 -4 1 6

Retains: results within specified thresholds, fold down for details.

Retains * p0 p25 p50 p75 p90 p99 p100 Samples
main 1292 1293 1293 1293 1293 1293 1293 355
pull_request 1292 1293 1293 1293 1293 1293 1293 361
Δ 0 0 0 0 0 0 0 6
Improvement % 0 0 0 0 0 0 0 6

@dduan dduan merged commit 5d61c18 into main Dec 30, 2025
16 checks passed
@dduan dduan deleted the dd/combine-parser-code branch December 30, 2025 18:45
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