Fix #545: no need to make members of static classes static.

Otherwise we will need to rewrite references to `This` of class be references on ModuleVal.
This is less efficient(instead of calling method statically known to be final, you have virtual call) and less jvm-friendly,
as needs additional instructions to get to ModuleVal.

1 files changed, 7 insertions, 1 deletions

diff --git a/src/dotty/tools/dotc/transform/LambdaLift.scala b/src/dotty/tools/dotc/transform/LambdaLift.scala
index 7be0a12ab..bac088928 100644
--- a/src/dotty/tools/dotc/transform/LambdaLift.scala
+++ b/src/dotty/tools/dotc/transform/LambdaLift.scala
@@ -288,7 +288,13 @@ class LambdaLift extends MiniPhase with IdentityDenotTransformer { thisTransform
     private def liftLocals()(implicit ctx: Context): Unit = {
       for ((local, lOwner) <- liftedOwner) {
         val (newOwner, maybeStatic) =
-          if (lOwner is Package) (local.topLevelClass, JavaStatic)
+          if (lOwner is Package)  {
+            println(s"lifting $local encl class ${local.enclosingClass}")
+            if (local.enclosingClass.isStatic) // member of a static object
+              (local.enclosingClass, EmptyFlags)
+            else
+              (local.topLevelClass, JavaStatic)
+          }
           else (lOwner, EmptyFlags)
         local.copySymDenotation(
           owner = newOwner,