rorygraves / scalac_perf

The Scala programming language
http://www.scala-lang.org/
16 stars 3 forks source link

phase packageobjects does too much? #43

Closed hrhino closed 6 years ago

hrhino commented 6 years ago
diff --git a/src/compiler/scala/tools/nsc/typechecker/Analyzer.scala b/src/compiler/scala/tools/nsc/typechecker/Analyzer.scala
index ccd414cc45..6442f0dfce 100644
--- a/src/compiler/scala/tools/nsc/typechecker/Analyzer.scala
+++ b/src/compiler/scala/tools/nsc/typechecker/Analyzer.scala
@@ -62,8 +62,8 @@ trait Analyzer extends AnyRef
             if (tree.symbol.name == nme.PACKAGEkw) {
               openPackageModule(tree.symbol, tree.symbol.owner)
             }
-          case ClassDef(_, _, _, _) => () // make it fast
-          case _ => super.traverse(tree)
+          case PackageDef(_, _) => super.traverse(tree)
+          case _ => ()
         }
       }
hrhino commented 6 years ago

after investigation, it doesn't, really.