merge from master

Author: Niklas Vest

.github/workflows/ci.yaml

@@ -15,6 +15,9 @@ jobs:
     container: lampepfl/dotty:2020-04-24
 
     steps:
+      - name: Set JDK 11 as default
+        run: echo "::add-path::/usr/lib/jvm/java-11-openjdk-amd64/bin"
+
       - name: Checkout cleanup script
         uses: actions/checkout@v2
 
@@ -55,6 +58,9 @@ jobs:
     container: lampepfl/dotty:2020-04-24
 
     steps:
+      - name: Set JDK 11 as default
+        run: echo "::add-path::/usr/lib/jvm/java-11-openjdk-amd64/bin"
+
       - name: Checkout cleanup script
         uses: actions/checkout@v2
 
@@ -174,7 +180,7 @@ jobs:
       - name: Test
         run: ./project/scripts/sbt sbt-dotty/scripted
 
-  test_java11:
+  test_java8:
     runs-on: self-hosted
     container: lampepfl/dotty:2020-04-24
     if: (
@@ -184,6 +190,9 @@ jobs:
         github.event_name == 'schedule'
 
     steps:
+      - name: Set JDK 8 as default
+        run: echo "::add-path::/usr/lib/jvm/java-1.8.0-openjdk-amd64/bin"
+
       - name: Checkout cleanup script
         uses: actions/checkout@v2
 
@@ -215,14 +224,12 @@ jobs:
           restore-keys: ${{ runner.os }}-general-
 
       - name: Test
-        run: |
-          export PATH="/usr/lib/jvm/java-11-openjdk-amd64/bin:$PATH"
-          ./project/scripts/sbt ";compile ;test"
+        run: ./project/scripts/sbt ";compile ;test"
 
   publish_nightly:
     runs-on: self-hosted
     container: lampepfl/dotty:2020-04-24
-    needs: [test, test_bootstrapped, community_build, test_sbt, test_java11]
+    needs: [test, test_bootstrapped, community_build, test_sbt, test_java8]
     if: github.event_name == 'schedule'
     env:
       NIGHTLYBUILD: yes
@@ -323,7 +330,7 @@ jobs:
   publish_release:
     runs-on: self-hosted
     container: lampepfl/dotty:2020-04-24
-    needs: [test, test_bootstrapped, community_build, test_sbt, test_java11]
+    needs: [test, test_bootstrapped, community_build, test_sbt, test_java8]
     if: github.event_name == 'push' &&
         startsWith(github.event.ref, 'refs/tags/') &&
         !startsWith(github.event.ref, 'refs/tags/sbt-dotty-')
@@ -475,7 +482,7 @@ jobs:
   publish_sbt_release:
     runs-on: self-hosted
     container: lampepfl/dotty:2020-04-24
-    needs: [test, test_bootstrapped, community_build, test_sbt, test_java11]
+    needs: [test, test_bootstrapped, community_build, test_sbt, test_java8]
     if: github.event_name == 'push' &&
         startsWith(github.event.ref, 'refs/tags/sbt-dotty-')
 

community-build/community-projects/dotty-cps-async

@@ -361,39 +361,39 @@ class CommunityBuildTest:
     }
   }
 
-  @Test def intent = projects.intent.run()
   @Test def algebra = projects.algebra.run()
+  @Test def betterfiles = projects.betterfiles.run()
+  @Test def dottyCpsAsync = projects.dottyCpsAsync.run()
+  @Test def effpi = projects.effpi.run()
+  @Test def endpoints = projects.endpoints.run()
+  @Test def fastparse = projects.fastparse.run()
+  @Test def geny = projects.geny.run()
+  @Test def intent = projects.intent.run()
+  @Test def minitest = projects.minitest.run()
+  @Test def munit = projects.munit.run()
+  @Test def oslib = projects.oslib.run()
+  // @Test def oslibWatch = projects.oslibWatch.run()
   @Test def scalacheck = projects.scalacheck.run()
+  @Test def scalap = projects.scalap.run()
+  @Test def scalaParserCombinators = projects.scalaParserCombinators.run()
+  @Test def ScalaPB = projects.ScalaPB.run()
   @Test def scalatest = projects.scalatest.run()
   @Test def scalatestplusScalacheck = projects.scalatestplusScalacheck.run()
   @Test def scalaXml = projects.scalaXml.run()
+  @Test def scalaz = projects.scalaz.run()
+  @Test def scodec = projects.scodec.run()
+  @Test def scodecBits = projects.scodecBits.run()
+  @Test def sconfig = projects.sconfig.run()
   @Test def scopt = projects.scopt.run()
-  @Test def scalap = projects.scalap.run()
-  @Test def squants = projects.squants.run()
-  @Test def betterfiles = projects.betterfiles.run()
-  @Test def ScalaPB = projects.ScalaPB.run()
-  @Test def minitest = projects.minitest.run()
-  @Test def fastparse = projects.fastparse.run()
-  @Test def utest = projects.utest.run()
+  @Test def shapeless = projects.shapeless.run()
   @Test def sourcecode = projects.sourcecode.run()
-  @Test def oslib = projects.oslib.run()
+  @Test def squants = projects.squants.run()
+  @Test def stdLib213 = projects.stdLib213.run()
   @Test def ujson = projects.ujson.run()
   // @Test def upickle = projects.upickle.run()
-  // @Test def oslibWatch = projects.oslibWatch.run()
-  @Test def geny = projects.geny.run()
-  @Test def stdLib213 = projects.stdLib213.run()
-  @Test def shapeless = projects.shapeless.run()
+  @Test def utest = projects.utest.run()
   @Test def xmlInterpolator = projects.xmlInterpolator.run()
-  @Test def effpi = projects.effpi.run()
-  @Test def sconfig = projects.sconfig.run()
   @Test def zio = projects.zio.run()
-  @Test def munit = projects.munit.run()
-  @Test def scodecBits = projects.scodecBits.run()
-  @Test def scodec = projects.scodec.run()
-  @Test def scalaParserCombinators = projects.scalaParserCombinators.run()
-  @Test def dottyCpsAsync = projects.dottyCpsAsync.run()
-  @Test def scalaz = projects.scalaz.run()
-  @Test def endpoints = projects.endpoints.run()
 end CommunityBuildTest
 
 class TestCategory

community-build/community-projects/scalatest

@@ -364,9 +364,13 @@ trait BCodeBodyBuilder extends BCodeSkelBuilder {
           }
           else {
             mnode.visitVarInsn(asm.Opcodes.ALOAD, 0)
-            generatedType =
-              if (tree.symbol == defn.ArrayClass) ObjectReference
-              else classBTypeFromSymbol(claszSymbol)
+            // When compiling Array.scala, the constructor invokes `Array.this.super.<init>`. The expectedType
+            // is `[Object` (computed by typeToBType, the type of This(Array) is `Array[T]`). If we would set
+            // the generatedType to `Array` below, the call to adapt at the end would fail. The situation is
+            // similar for primitives (`I` vs `Int`).
+            if (tree.symbol != defn.ArrayClass && !tree.symbol.isPrimitiveValueClass) {
+              generatedType = classBTypeFromSymbol(claszSymbol)
+            }
           }
 
         case DesugaredSelect(Ident(nme.EMPTY_PACKAGE), module) =>
@@ -512,9 +516,16 @@ trait BCodeBodyBuilder extends BCodeSkelBuilder {
 
         case ClazzTag   =>
           val tp = toTypeKind(const.typeValue)
-          // classOf[Int] is transformed to Integer.TYPE by ClassOf
-          assert(!tp.isPrimitive, s"expected class type in classOf[T], found primitive type $tp")
-          mnode.visitLdcInsn(tp.toASMType)
+          if tp.isPrimitive then
+            val boxedClass = boxedClassOfPrimitive(tp.asPrimitiveBType)
+            mnode.visitFieldInsn(
+              asm.Opcodes.GETSTATIC,
+              boxedClass.internalName,
+              "TYPE", // field name
+              jlClassRef.descriptor
+            )
+          else
+            mnode.visitLdcInsn(tp.toASMType)
 
         case EnumTag   =>
           val sym       = const.symbolValue
@@ -703,33 +714,18 @@ trait BCodeBodyBuilder extends BCodeSkelBuilder {
             if (t.symbol ne defn.Object_synchronized) genTypeApply(t)
             else genSynchronized(app, expectedType)
 
-        case Apply(fun @ DesugaredSelect(Super(_, _), _), args) =>
-          def initModule(): Unit = {
-            // we initialize the MODULE$ field immediately after the super ctor
-            if (!isModuleInitialized &&
-              jMethodName == INSTANCE_CONSTRUCTOR_NAME &&
-              fun.symbol.javaSimpleName == INSTANCE_CONSTRUCTOR_NAME &&
-              claszSymbol.isStaticModuleClass) {
-              isModuleInitialized = true
-              mnode.visitVarInsn(asm.Opcodes.ALOAD, 0)
-              mnode.visitFieldInsn(
-                asm.Opcodes.PUTSTATIC,
-                thisName,
-                str.MODULE_INSTANCE_FIELD,
-                "L" + thisName + ";"
-              )
-            }
-          }
+        case Apply(fun @ DesugaredSelect(Super(superQual, _), _), args) =>
           // 'super' call: Note: since constructors are supposed to
           // return an instance of what they construct, we have to take
           // special care. On JVM they are 'void', and Scala forbids (syntactically)
           // to call super constructors explicitly and/or use their 'returned' value.
           // therefore, we can ignore this fact, and generate code that leaves nothing
           // on the stack (contrary to what the type in the AST says).
-          mnode.visitVarInsn(asm.Opcodes.ALOAD, 0)
+
+          // scala/bug#10290: qual can be `this.$outer()` (not just `this`), so we call genLoad (not just ALOAD_0)
+          genLoad(superQual)
           genLoadArguments(args, paramTKs(app))
           generatedType = genCallMethod(fun.symbol, InvokeStyle.Super, app.span)
-          initModule()
 
         // 'new' constructor call: Note: since constructors are
         // thought to return an instance of what they construct,

community-build/community-projects/scodec-bits

@@ -29,6 +29,7 @@ trait BCodeIdiomatic {
     case "jvm-1.6"     => asm.Opcodes.V1_6
     case "jvm-1.7"     => asm.Opcodes.V1_7
     case "jvm-1.8"     => asm.Opcodes.V1_8
+    case "jvm-9"       => asm.Opcodes.V9
   }
 
   lazy val majorVersion: Int = (classfileVersion & 0xFF)