diff options
author | Paul Phillips <paulp@improving.org> | 2010-12-22 20:55:10 +0000 |
---|---|---|
committer | Paul Phillips <paulp@improving.org> | 2010-12-22 20:55:10 +0000 |
commit | d519e34eb5734c7490bf88262e3abb6a56af1286 (patch) | |
tree | 32a733715d4600809fd37761ecfd855ffa1b1aa7 /src/compiler | |
parent | db4bf36110a922bd41a900a90fb2c158280d027b (diff) | |
download | scala-d519e34eb5734c7490bf88262e3abb6a56af1286.tar.gz scala-d519e34eb5734c7490bf88262e3abb6a56af1286.tar.bz2 scala-d519e34eb5734c7490bf88262e3abb6a56af1286.zip |
A couple caches knock a full 20% off the time s...
A couple caches knock a full 20% off the time spent in genjvm. Review by
dragos.
Diffstat (limited to 'src/compiler')
-rw-r--r-- | src/compiler/scala/tools/nsc/backend/jvm/GenJVM.scala | 8 |
1 files changed, 6 insertions, 2 deletions
diff --git a/src/compiler/scala/tools/nsc/backend/jvm/GenJVM.scala b/src/compiler/scala/tools/nsc/backend/jvm/GenJVM.scala index 2bc501cea0..b2dc217708 100644 --- a/src/compiler/scala/tools/nsc/backend/jvm/GenJVM.scala +++ b/src/compiler/scala/tools/nsc/backend/jvm/GenJVM.scala @@ -1726,11 +1726,15 @@ abstract class GenJVM extends SubComponent with GenJVMUtil with GenAndroid { ////////////////////// Utilities //////////////////////// - override def javaName(sym: Symbol): String = { + private val javaNameCache = mutable.HashMap[Symbol, String]() + override def javaName(sym: Symbol): String = javaNameCache.getOrElseUpdate(sym, { if (sym.isClass && !sym.rawowner.isPackageClass && !sym.isModuleClass) innerClasses = innerClasses + sym super.javaName(sym) - } + }) + + private val javaTypeCache = mutable.HashMap[Symbol, JType]() + override def javaType(sym: Symbol): JType = javaTypeCache.getOrElseUpdate(sym, super.javaType(sym)) /** Calls to methods in 'sym' need invokeinterface? */ def needsInterfaceCall(sym: Symbol): Boolean = { |