summaryrefslogtreecommitdiff
path: root/dbuild-meta.json
diff options
context:
space:
mode:
authorJason Zaugg <jzaugg@gmail.com>2016-02-09 22:12:26 +1000
committerJason Zaugg <jzaugg@gmail.com>2016-02-09 22:12:26 +1000
commitc3fcd4e5b0c8f03862ecf4f92fd43cf2fb8f5066 (patch)
treeb7f6f0fbbbf514687bc20e18c94317cb9905c673 /dbuild-meta.json
parentef4ed496c59e4b54266aaf20db917fb41bb9f1e4 (diff)
downloadscala-c3fcd4e5b0c8f03862ecf4f92fd43cf2fb8f5066.tar.gz
scala-c3fcd4e5b0c8f03862ecf4f92fd43cf2fb8f5066.tar.bz2
scala-c3fcd4e5b0c8f03862ecf4f92fd43cf2fb8f5066.zip
Micro optimise Symbol#fullName
The old approach of recursively calling `fullNameAsName` creates a lot of garbage for intermediate results, in addition to needless interning of those results into the name table. This commit instead creates a string buffer of the correct capacity and writes the component names directly into this. I compared old and new approaches and this shows a 2x speedup. ``` scala> val th = ichi.bench.Thyme.warmed(verbose = print) th: ichi.bench.Thyme = ichi.bench.Thyme@1643e817 scala> val w_old = th.Warm(sym.fullNameAsNameOld('.')) w_old: th.Warm[$r.intp.global.Name] = ichi.bench.Thyme$Warm@7a8d001b scala> val w_new = th.Warm(sym.fullNameAsName('.')) w_new: th.Warm[$r.intp.global.Name] = ichi.bench.Thyme$Warm@1ec14586 scala> th.pbenchOffWarm("", x => println(x))(w_old, 10, "old")(w_new, 10, "new") Benchmark comparison (in 4.084 s) old vs new Significantly different (p ~= 0) Time ratio: 0.53572 95% CI 0.51618 - 0.55525 (n=20) old 64.54 ns 95% CI 62.41 ns - 66.67 ns new 34.57 ns 95% CI 34.04 ns - 35.11 ns res3: $r.intp.global.Name = scala.collection.parallel.mutable.ParSeq ``` It is still expensive enough that we should still consider caching. The call to full name in `classBTypeFromSymbol` in the new backed is a prime candidate for optimization.
Diffstat (limited to 'dbuild-meta.json')
0 files changed, 0 insertions, 0 deletions