search

kkebo / zyphy

Zyphy is (or will be) a fast web browser engine written in Swift.
MIT License
4 stars 1 forks source link

perf: make `Tag` non-copyable #105

Closed kkebo closed 1 month ago

github-actions[bot] commented 1 month ago

Code Metrics Report

main (530f1ae) #105 (d696bf8) +/-
Coverage 85.3% 85.2% -0.2%
Details ``` diff | | main (530f1ae) | #105 (d696bf8) | +/- | |-----------|----------------|----------------|-------| - | Coverage | 85.3% | 85.2% | -0.2% | | Files | 9 | 9 | 0 | | Lines | 1588 | 1596 | +8 | + | Covered | 1356 | 1360 | +4 | ```

Code coverage of files in pull request scope (100.0% → 100.0%)

Files Coverage +/-
Sources/Tokenizer/Tag.swift 100.0% 0.0%

Reported by octocov

kkebo commented 1 month ago

Comparing results between 'main' and 'Current_run'

Host 'Raspberry-beetle' with 4 'aarch64' processors with 7 GB memory, running:
#1 SMP PREEMPT Debian 1:6.6.47-1+rpt1 (2024-09-02)

MyBenchmark

lipsum 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 | 24 | 24 | 24 | 24 | 24 | 25 | 25 | 100 | | Current_run | 24 | 24 | 24 | 24 | 24 | 24 | 24 | 100 | | Δ | 0 | 0 | 0 | 0 | 0 | -1 | -1 | 0 | | Improvement % | 0 | 0 | 0 | 0 | 0 | 4 | 4 | 0 |

lipsum-zh 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 | 3010 | 3017 | 3023 | 3033 | 3039 | 3142 | 3144 | 100 | | Current_run | 3036 | 3049 | 3056 | 3064 | 3072 | 3101 | 3183 | 100 | | Δ | 26 | 32 | 33 | 31 | 33 | -41 | 39 | 0 | | Improvement % | -1 | -1 | -1 | -1 | -1 | 1 | -1 | 0 |

medium-fragment 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 | 71 | 71 | 71 | 71 | 71 | 72 | 72 | 100 | | Current_run | 69 | 69 | 69 | 70 | 71 | 71 | 71 | 100 | | Δ | -2 | -2 | -2 | -1 | 0 | -1 | -1 | 0 | | Improvement % | 3 | 3 | 3 | 1 | 0 | 1 | 1 | 0 |

small-fragment 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 | 7374 | 7414 | 7422 | 7434 | 7438 | 7475 | 7748 | 100 | | Current_run | 7498 | 7516 | 7528 | 7586 | 7619 | 7660 | 7672 | 100 | | Δ | 124 | 102 | 106 | 152 | 181 | 185 | -76 | 0 | | Improvement % | -2 | -1 | -1 | -2 | -2 | -2 | 1 | 0 |

strong 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 | 35 | 35 | 35 | 35 | 35 | 35 | 35 | 100 | | Current_run | 36 | 36 | 36 | 36 | 36 | 37 | 37 | 100 | | Δ | 1 | 1 | 1 | 1 | 1 | 2 | 2 | 0 | | Improvement % | -3 | -3 | -3 | -3 | -3 | -6 | -6 | 0 |

tiny-fragment 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 | 710 | 711 | 712 | 713 | 715 | 728 | 736 | 100 | | Current_run | 728 | 733 | 735 | 738 | 740 | 753 | 973 | 100 | | Δ | 18 | 22 | 23 | 25 | 25 | 25 | 237 | 0 | | Improvement % | -3 | -3 | -3 | -4 | -3 | -3 | -32 | 0 |

github-actions[bot] commented 1 month ago

Code Metrics Report

main (ea644aa) #105 (5dcbe87) +/-
Coverage 85.3% 85.4% +0.0%
Details ``` diff | | main (ea644aa) | #105 (5dcbe87) | +/- | |-----------|----------------|----------------|-------| + | Coverage | 85.3% | 85.4% | +0.0% | | Files | 9 | 9 | 0 | | Lines | 1598 | 1593 | -5 | - | Covered | 1364 | 1361 | -3 | ```

Code coverage of files in pull request scope (100.0% → 100.0%)

Files Coverage +/-
Sources/Tokenizer/Tag.swift 100.0% 0.0%

Reported by octocov