Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

perf: warmup V8 symbols generation of the performance map #15

Merged
merged 1 commit into from
Jul 3, 2023

Conversation

adriencaccia
Copy link
Member

No description provided.

@codspeed-hq
Copy link

codspeed-hq bot commented Jun 29, 2023

CodSpeed Performance Report

Merging #15 will degrade performances by 21.16%

Comparing feat/add-warmup (3630e0e) with main (38de55d)

Summary

🔥 61 improvements
❌ 1 regressions
✅ 2 untouched benchmarks

⚠️ Please fix the performance issues or acknowledge them on CodSpeed.

Benchmarks breakdown

Benchmark main feat/add-warmup Change
🔥 switch 1 222.6 µs 16.9 µs ×13
🔥 switch 2 239.5 µs 17.6 µs ×14
🔥 RegExp#test 210.2 µs 20.5 µs ×10
🔥 String#indexOf 201.2 µs 14.2 µs ×14
🔥 switch 1 223.6 µs 16.7 µs ×13
🔥 switch 2 240.2 µs 17.4 µs ×14
🔥 switch 1 239.8 µs 18.7 µs ×13
🔥 switch 2 254 µs 20.2 µs ×13
🔥 test_recursive_fibo_10 757.1 µs 170.1 µs ×4.5
🔥 test_recursive_fibo_20 14.9 ms 8.8 ms +69.92%
🔥 test_recursive_cached_fibo_10 3,028.4 µs 68.1 µs ×44
🔥 test_recursive_cached_fibo_20 314.6 µs 105.9 µs ×3
🔥 test_recursive_cached_fibo_30 326 µs 152.1 µs ×2.1
🔥 test_iterative_fibo_10 2,133.2 µs 42.8 µs ×50
🔥 test_iterative_fibo_100 336.4 µs 74.3 µs ×4.5
🔥 test sync baz 10 837.2 µs 40.4 µs ×21
🔥 test sync baz 100 259.4 µs 62.9 µs ×4.1
🔥 test async baz 10 3.6 ms 2.2 ms +64.4%
🔥 test async baz 100 2.3 ms 1.2 ms +92.17%
🔥 test_recursive_fibo_10 1,301.4 µs 94 µs ×14
🔥 test_recursive_cached_fibo_10 2,148.7 µs 68.2 µs ×31
🔥 test_recursive_cached_fibo_20 315.6 µs 106.1 µs ×3
🔥 test_recursive_cached_fibo_30 326.8 µs 152.3 µs ×2.1
🔥 test_iterative_fibo_10 543.9 µs 43 µs ×13
🔥 test_iterative_fibo_100 274.3 µs 74.5 µs ×3.7
🔥 test sync baz 10 1,285.6 µs 40.5 µs ×32
🔥 test sync baz 100 268.7 µs 63.1 µs ×4.3
🔥 test async baz 10 7.8 ms 1.2 ms ×6.3
test async baz 100 749.8 µs 951.1 µs -21.16%
🔥 switch 1 223.9 µs 16.8 µs ×13
🔥 switch 2 240.7 µs 17.5 µs ×14
🔥 switch 1 240.1 µs 18.7 µs ×13
🔥 switch 2 1,822.1 µs 20.3 µs ×90
🔥 RegExp#test 206.7 µs 20.5 µs ×10
🔥 String#indexOf 197.7 µs 14.1 µs ×14
🔥 RegExp#test 210.5 µs 20.7 µs ×10
🔥 String#indexOf 203.1 µs 14.2 µs ×14
🔥 test_recursive_fibo_10 1,345.3 µs 161.6 µs ×8.3
🔥 test_recursive_cached_fibo_10 848.7 µs 59.7 µs ×14
🔥 test_recursive_cached_fibo_20 308.7 µs 97.5 µs ×3.2
🔥 test_recursive_cached_fibo_30 320.1 µs 143.7 µs ×2.2
🔥 test_iterative_fibo_10 1,291.2 µs 34.3 µs ×38
🔥 test_iterative_fibo_100 267.4 µs 65.8 µs ×4.1
🔥 test sync baz 10 1,641.3 µs 31.8 µs ×52
🔥 test sync baz 100 253.8 µs 31.1 µs ×8.2
🔥 test async baz 10 1,428.1 µs 105.6 µs ×14
🔥 test async baz 100 338.2 µs 86.2 µs ×3.9
🔥 test_recursive_fibo_10 751.8 µs 161.8 µs ×4.6
🔥 test_recursive_fibo_20 18.7 ms 14.3 ms +30.4%
🔥 test_recursive_cached_fibo_10 847 µs 59.9 µs ×14
🔥 test_recursive_cached_fibo_20 308.5 µs 97.7 µs ×3.2
🔥 test_recursive_cached_fibo_30 903.8 µs 143.8 µs ×6.3
🔥 test_iterative_fibo_10 538 µs 34.4 µs ×16
🔥 test_iterative_fibo_100 267.3 µs 65.9 µs ×4.1
🔥 test sync baz 10 1,273.2 µs 31.9 µs ×40
🔥 test sync baz 100 253.2 µs 54.4 µs ×4.7
🔥 test async baz 10 1,304.7 µs 105.5 µs ×12
🔥 test async baz 100 337.7 µs 135.6 µs ×2.5
🔥 RegExp#test 210.9 µs 20.5 µs ×10
🔥 String#indexOf 733.3 µs 14.2 µs ×52
🔥 RegExp#test 211.6 µs 20.4 µs ×10
🔥 String#indexOf 201.3 µs 14.1 µs ×14

@art049 art049 force-pushed the feat/add-warmup branch 2 times, most recently from d3770a6 to 3630e0e Compare July 3, 2023 14:08
@art049 art049 merged commit 3630e0e into main Jul 3, 2023
@art049 art049 deleted the feat/add-warmup branch July 3, 2023 14:54
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