Avoid Context#clone #6029
Avoid Context#clone #6029
Conversation
|
test performance please |
|
performance test scheduled: 1 job(s) in queue, 1 running. |
|
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/6029/ to see the changes. Benchmarks is based on merging with master (5f5b5b0) |
|
test performance please |
|
performance test scheduled: 1 job(s) in queue, 1 running. |
|
performance test failed: Please check http://lamppc37.epfl.ch:8000/pull-6029-03-07-16.21.out for more information |
odersky
force-pushed
the
try-optimize-ctx
branch
from
13c4e2c to
02ff67a
Compare
|
test performance please |
|
performance test scheduled: 2 job(s) in queue, 1 running. |
|
Performance test finished successfully: Visit http://dotty-bench.epfl.ch/6029/ to see the changes. Benchmarks is based on merging with master (7d30a83) |
|
We are in the noise wrt efficiency. I tested twice, the first time stdlib made a jump up, and the second time an equally large jump down. The other tests are more or less flat. But clarity is a clear win, so I am going to merge this. |
Creating a context with a regular allocation is clearer.
Let's find out whether it is more efficient or not.