Skip to content

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

Closed
wants to merge 10 commits into from

Conversation

odersky
Copy link
Contributor

@odersky odersky commented Sep 27, 2020

Break some often-called large methods into smaller pieces while also optimizing order of pattern tests

@odersky
Copy link
Contributor Author

odersky commented Sep 27, 2020

test performance please

@dottybot
Copy link
Member

performance test scheduled: 1 job(s) in queue, 0 running.

@dottybot
Copy link
Member

Performance test finished successfully:

Visit http://dotty-bench.epfl.ch/9896/ to see the changes.

Benchmarks is based on merging with master (d084b8b)

@odersky odersky force-pushed the optimize-large-methods branch from 9498b06 to 673b9cb Compare September 27, 2020 14:53
@odersky
Copy link
Contributor Author

odersky commented Sep 27, 2020

test performance please

@dottybot
Copy link
Member

performance test scheduled: 1 job(s) in queue, 0 running.

@dottybot
Copy link
Member

Performance test finished successfully:

Visit http://dotty-bench.epfl.ch/9896/ to see the changes.

Benchmarks is based on merging with master (d084b8b)

@odersky odersky force-pushed the optimize-large-methods branch from 673b9cb to 4570fff Compare October 2, 2020 14:11
@odersky
Copy link
Contributor Author

odersky commented Oct 2, 2020

test performance please

@dottybot
Copy link
Member

dottybot commented Oct 2, 2020

performance test scheduled: 2 job(s) in queue, 1 running.

@odersky
Copy link
Contributor Author

odersky commented Oct 2, 2020

test performance please

@dottybot
Copy link
Member

dottybot commented Oct 2, 2020

performance test scheduled: 3 job(s) in queue, 1 running.

@dottybot
Copy link
Member

dottybot commented Oct 2, 2020

Performance test finished successfully:

Visit http://dotty-bench.epfl.ch/9896/ to see the changes.

Benchmarks is based on merging with master (55e378d)

@dottybot
Copy link
Member

dottybot commented Oct 2, 2020

Performance test finished successfully:

Visit http://dotty-bench.epfl.ch/9896/ to see the changes.

Benchmarks is based on merging with master (371d8f5)

@odersky
Copy link
Contributor Author

odersky commented Oct 3, 2020

test performance please

@dottybot
Copy link
Member

dottybot commented Oct 3, 2020

performance test scheduled: 1 job(s) in queue, 0 running.

@dottybot
Copy link
Member

dottybot commented Oct 3, 2020

Performance test finished successfully:

Visit http://dotty-bench.epfl.ch/9896/ to see the changes.

Benchmarks is based on merging with master (371d8f5)

@odersky
Copy link
Contributor Author

odersky commented Oct 6, 2020

I have moved what's worth keeping to #9867

@odersky odersky closed this Oct 6, 2020
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