-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Optimize core and frontend #9867
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
Conversation
test performance please |
performance test scheduled: 1 job(s) in queue, 1 running. |
do buf += sym | ||
for bc <- clazz.baseClasses; sym <- bc.info.decls.toList do | ||
if sym.is(DeferredTerm) && !isImplemented(sym) && !ignoreDeferred(sym) | ||
buf += sym |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
then
is required after #9859 -- the benchmarks fail due to this.
performance test failed: Please check http://lamppc37.epfl.ch:8000/pull-9867-09-24-18.05.out for more information |
test performance please |
performance test scheduled: 7 job(s) in queue, 1 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9867/ to see the changes. Benchmarks is based on merging with master (7ee3a20) |
88e012c
to
8205569
Compare
test performance please |
performance test scheduled: 5 job(s) in queue, 1 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9867/ to see the changes. Benchmarks is based on merging with master (371d8f5) |
Make all orElse methods that take a call-by-name argument inline methods.
8205569
to
4c6f317
Compare
test performance please |
performance test scheduled: 2 job(s) in queue, 1 running. |
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/9867/ to see the changes. Benchmarks is based on merging with master (63dd4f9) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
No description provided.