-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Optimize large methods #9896
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
Optimize large methods #9896
Conversation
test performance please |
performance test scheduled: 1 job(s) in queue, 0 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9896/ to see the changes. Benchmarks is based on merging with master (d084b8b) |
9498b06
to
673b9cb
Compare
test performance please |
performance test scheduled: 1 job(s) in queue, 0 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9896/ to see the changes. Benchmarks is based on merging with master (d084b8b) |
673b9cb
to
4570fff
Compare
test performance please |
performance test scheduled: 2 job(s) in queue, 1 running. |
Order first 10 entries according to allocation frequency
test performance please |
performance test scheduled: 3 job(s) in queue, 1 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9896/ to see the changes. Benchmarks is based on merging with master (55e378d) |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9896/ to see the changes. Benchmarks is based on merging with master (371d8f5) |
test performance please |
performance test scheduled: 1 job(s) in queue, 0 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9896/ to see the changes. Benchmarks is based on merging with master (371d8f5) |
I have moved what's worth keeping to #9867 |
Break some often-called large methods into smaller pieces while also optimizing order of pattern tests