kkebo / zyphy

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

refactor: use string literals #79

Closed kkebo closed 2 months ago

kkebo commented 4 months ago

Comparing results between 'main' and 'Current_run'

Host 'Brown-rhinoceros-beetle' with 8 'aarch64' processors with 7 GB memory, running:                                                                                                                                
#1 SMP PREEMPT_DYNAMIC Sat Jun 22 08:04:30 UTC 2024                                                                                                                                                                  

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 | 14 | 14 | 14 | 14 | 15 | 15 | 16 | 100 | | Current_run | 14 | 14 | 14 | 14 | 15 | 15 | 15 | 100 | | Δ | 0 | 0 | 0 | 0 | 0 | 0 | -1 | 0 | | Improvement % | 0 | 0 | 0 | 0 | 0 | 0 | 6 | 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 | 1549 | 1556 | 1564 | 1573 | 1581 | 1684 | 1690 | 100 | | Current_run | 1551 | 1558 | 1563 | 1606 | 1617 | 1689 | 1728 | 100 | | Δ | 2 | 2 | -1 | 33 | 36 | 5 | 38 | 0 | | Improvement % | 0 | 0 | 0 | -2 | -2 | 0 | -2 | 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 | 44 | 44 | 44 | 44 | 45 | 47 | 47 | 100 | | Current_run | 44 | 44 | 44 | 44 | 45 | 47 | 48 | 100 | | Δ | 0 | 0 | 0 | 0 | 0 | 0 | 1 | 0 | | Improvement % | 0 | 0 | 0 | 0 | 0 | 0 | -2 | 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 | 4596 | 4604 | 4616 | 4764 | 4776 | 4796 | 4799 | 100 | | Current_run | 4594 | 4604 | 4612 | 4674 | 4772 | 4940 | 4940 | 100 | | Δ | -2 | 0 | -4 | -90 | -4 | 144 | 141 | 0 | | Improvement % | 0 | 0 | 0 | 2 | 0 | -3 | -3 | 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 | 20 | 20 | 20 | 20 | 21 | 22 | 22 | 100 | | Current_run | 21 | 21 | 21 | 21 | 21 | 22 | 22 | 100 | | Δ | 1 | 1 | 1 | 1 | 0 | 0 | 0 | 0 | | Improvement % | -5 | -5 | -5 | -5 | 0 | 0 | 0 | 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 | 464 | 467 | 483 | 538 | 573 | 579 | 580 | 100 | | Current_run | 465 | 468 | 483 | 501 | 503 | 544 | 544 | 100 | | Δ | 1 | 1 | 0 | -37 | -70 | -35 | -36 | 0 | | Improvement % | 0 | 0 | 0 | 7 | 12 | 6 | 6 | 0 |

github-actions[bot] commented 4 months ago

Code Metrics Report

main (697e587) #79 (25885ad) +/-
Coverage 89.2% 89.3% +0.1%
Details ``` diff | | main (697e587) | #79 (25885ad) | +/- | |-----------|----------------|---------------|-------| + | Coverage | 89.2% | 89.3% | +0.1% | | Files | 9 | 9 | 0 | | Lines | 1609 | 1619 | +10 | + | Covered | 1435 | 1445 | +10 | ```

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

Files Coverage +/-
Sources/Tokenizer/BufferQueue.swift 100.0% 0.0%
Sources/Tokenizer/CharRefTokenizer.swift 97.8% 0.0%
Sources/Tokenizer/Tokenizer.swift 87.4% +0.1%

Reported by octocov

github-actions[bot] commented 4 months ago

Code Metrics Report

main (697e587) #79 (e3b95b3) +/-
Coverage 89.2% 89.3% +0.1%
Details ``` diff | | main (697e587) | #79 (e3b95b3) | +/- | |-----------|----------------|---------------|-------| + | Coverage | 89.2% | 89.3% | +0.1% | | Files | 9 | 9 | 0 | | Lines | 1609 | 1619 | +10 | + | Covered | 1435 | 1445 | +10 | ```

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

Files Coverage +/-
Sources/Tokenizer/BufferQueue.swift 100.0% 0.0%
Sources/Tokenizer/CharRefTokenizer.swift 97.8% 0.0%
Sources/Tokenizer/Tokenizer.swift 87.4% +0.1%

Reported by octocov

github-actions[bot] commented 4 months ago

Code Metrics Report

main (cd73634) #79 (a0583e6) +/-
Coverage 89.1% 89.2% +0.0%
Details ``` diff | | main (cd73634) | #79 (a0583e6) | +/- | |-----------|----------------|---------------|-------| + | Coverage | 89.1% | 89.2% | +0.0% | | Files | 9 | 9 | 0 | | Lines | 1608 | 1612 | +4 | + | Covered | 1434 | 1438 | +4 | ```

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

Files Coverage +/-
Sources/Tokenizer/BufferQueue.swift 100.0% 0.0%
Sources/Tokenizer/CharRefTokenizer.swift 97.7% 0.0%

Reported by octocov

github-actions[bot] commented 3 months ago

Code Metrics Report

main (6387067) #79 (6200914) +/-
Coverage 85.1% 85.1% +0.0%
Details ``` diff | | main (6387067) | #79 (6200914) | +/- | |-----------|----------------|---------------|-------| + | Coverage | 85.1% | 85.1% | +0.0% | | Files | 9 | 9 | 0 | | Lines | 1608 | 1612 | +4 | + | Covered | 1369 | 1373 | +4 | ```

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

Files Coverage +/-
Sources/Tokenizer/BufferQueue.swift 100.0% 0.0%
Sources/Tokenizer/CharRefTokenizer.swift 97.7% 0.0%

Reported by octocov

github-actions[bot] commented 3 months ago

Code Metrics Report

main (b919988) #79 (4d821ef) +/-
Coverage 85.1% 85.2% +0.0%
Details ``` diff | | main (b919988) | #79 (4d821ef) | +/- | |-----------|----------------|---------------|-------| + | Coverage | 85.1% | 85.2% | +0.0% | | Files | 9 | 9 | 0 | | Lines | 1612 | 1616 | +4 | + | Covered | 1373 | 1377 | +4 | ```

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

Files Coverage +/-
Sources/Tokenizer/BufferQueue.swift 100.0% 0.0%
Sources/Tokenizer/CharRefTokenizer.swift 97.7% 0.0%

Reported by octocov

github-actions[bot] commented 3 months ago

Code Metrics Report

main (0e8bce6) #79 (fee10eb) +/-
Coverage 85.1% 85.2% +0.0%
Details ``` diff | | main (0e8bce6) | #79 (fee10eb) | +/- | |-----------|----------------|---------------|-------| + | Coverage | 85.1% | 85.2% | +0.0% | | Files | 9 | 9 | 0 | | Lines | 1612 | 1616 | +4 | + | Covered | 1373 | 1377 | +4 | ```

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

Files Coverage +/-
Sources/Tokenizer/BufferQueue.swift 100.0% 0.0%
Sources/Tokenizer/CharRefTokenizer.swift 97.7% 0.0%

Reported by octocov

github-actions[bot] commented 3 months ago

Code Metrics Report

main (0e8bce6) #79 (97aaed1) +/-
Coverage 85.1% 84.8% -0.4%
Details ``` diff | | main (0e8bce6) | #79 (97aaed1) | +/- | |-----------|----------------|---------------|-------| - | Coverage | 85.1% | 84.8% | -0.4% | | Files | 9 | 9 | 0 | | Lines | 1612 | 1625 | +13 | + | Covered | 1373 | 1378 | +5 | ```

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

Files Coverage +/-
Sources/Tokenizer/CharRefTokenizer.swift 97.7% 0.0%
Sources/Tokenizer/Str.swift 46.1% -7.7%
Sources/Tokenizer/Tokenizer.swift 82.3% 0.0%

Reported by octocov

github-actions[bot] commented 2 months ago

Code Metrics Report

main (1a1ad5f) #79 (6dfd06e) +/-
Coverage 85.1% 84.8% -0.4%
Details ``` diff | | main (1a1ad5f) | #79 (6dfd06e) | +/- | |-----------|----------------|---------------|-------| - | Coverage | 85.1% | 84.8% | -0.4% | | Files | 9 | 9 | 0 | | Lines | 1612 | 1625 | +13 | + | Covered | 1373 | 1378 | +5 | ```

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

Files Coverage +/-
Sources/Tokenizer/CharRefTokenizer.swift 97.7% 0.0%
Sources/Tokenizer/Str.swift 46.1% -7.7%
Sources/Tokenizer/Tokenizer.swift 82.3% 0.0%

Reported by octocov