diff options
author | Eugene Burmako <xeno.by@gmail.com> | 2012-09-23 15:07:53 +0200 |
---|---|---|
committer | Eugene Burmako <xeno.by@gmail.com> | 2012-09-26 18:20:22 +0200 |
commit | 2fb507b849e2b0f86c2480bde95200f8ae30803d (patch) | |
tree | 11405d294f3d61c57eb0807d37be18adfdd92dea /test/files/run/reify_ann5.scala | |
parent | f7efb9505c0ede48a67fb6a256604ba3ca02bcb3 (diff) | |
download | scala-2fb507b849e2b0f86c2480bde95200f8ae30803d.tar.gz scala-2fb507b849e2b0f86c2480bde95200f8ae30803d.tar.bz2 scala-2fb507b849e2b0f86c2480bde95200f8ae30803d.zip |
SI-6277 fixes flags, annotations & privateWithin
`Symbol.getFlag`, 'Symbol.hasFlag`, `Symbol.hasAllFlags`, `Symbol.annotations`
and `Symbol.privateWithin` now trigger automatic initialization of symbols
if they are used in a runtime reflection universe and some other conditions
are met (see `Symbol.needsInitialize` for details).
As the performance testing in https://github.com/scala/scala/pull/1380 shows,
this commit introduces a ~2% performance regression of compilation speed.
Unfortunately all known solutions to the bug at hand (A, B & C - all of those)
introduce perf regressions (see the pull request linked above for details).
However we're under severe time pressure, so there's no more time to explore.
Therefore I suggest this is reasonable to accept this performance hit,
because we've just gained 6% from removing scala.reflect.base,
and even before that we were well within our performance goal for 2.10.0-final.
Diffstat (limited to 'test/files/run/reify_ann5.scala')
-rw-r--r-- | test/files/run/reify_ann5.scala | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/test/files/run/reify_ann5.scala b/test/files/run/reify_ann5.scala index 877360180c..d27be3b6d5 100644 --- a/test/files/run/reify_ann5.scala +++ b/test/files/run/reify_ann5.scala @@ -16,7 +16,6 @@ object Test extends App { // test 2: import and typecheck val toolbox = cm.mkToolBox() val ttree = toolbox.typeCheck(tree) - ttree.foreach(sub => if (sub.hasSymbol) sub.symbol.typeSignature) println(ttree.toString) // test 3: import and compile |