Cleanup

Author: Duhemm

compiler/src/dotty/tools/dotc/transform/SpecializeFunctions.scala

@@ -15,16 +15,6 @@ class SpecializeFunctions extends MiniPhaseTransform with InfoTransformer {
   import ast.tpd._
   val phaseName = "specializeFunctions"
 
-  private[this] var _blacklistedSymbols: List[Symbol] = _
-
-  private def blacklistedSymbols(implicit ctx: Context): List[Symbol] = {
-    if (_blacklistedSymbols eq null) _blacklistedSymbols = List(
-      ctx.getClassIfDefined("scala.math.Ordering").asClass.membersNamed("Ops".toTypeName).first.symbol
-    )
-
-    _blacklistedSymbols
-  }
-
   /** Transforms the type to include decls for specialized applys and replace
    *  the class parents with specialized versions.
    */
@@ -84,7 +74,7 @@ class SpecializeFunctions extends MiniPhaseTransform with InfoTransformer {
           dt.vparamss.head.map(_.symbol.info)
         )
 
-        val specializedApply = tree.symbol.enclosingClass.info.decls.lookup(specName)//member(specName).symbol
+        val specializedApply = tree.symbol.enclosingClass.info.decls.lookup(specName)
 
         if (specializedApply.exists) {
           val apply = specializedApply.asTerm

compiler/test/dotty/tools/dotc/transform/SpecializeFunctionsTests.scala

@@ -60,8 +60,8 @@ class SpecializeFunctionsTests extends DottyBytecodeTest {
         apps.length == 3,
         s"Wrong number of specialized applys, actual length: ${apps.length} - $apps"
       )
-      assert(apps.contains("apply"), "Func3 did not contain `apply` forwarder method")
-      assert(apps.contains("apply$mcIII$sp"), "Func3 did not contain specialized apply")
+      assert(apps.contains("apply"), "Func2 did not contain `apply` forwarder method")
+      assert(apps.contains("apply$mcIII$sp"), "Func2 did not contain specialized apply")
     }
   }
 
@@ -91,6 +91,7 @@ class SpecializeFunctionsTests extends DottyBytecodeTest {
          |}""".stripMargin
 
     checkBCode(source) { dir =>
+      // No specialization for Function1[Char, Int]
       assertBoxing("<init>", findClass("Test$", dir).methods)
     }
   }
@@ -136,6 +137,7 @@ class SpecializeFunctionsTests extends DottyBytecodeTest {
          |}""".stripMargin
 
     checkBCode(source) { dir =>
+      // No specialization for Function2[Char, Char, Char]
       assertBoxing("<init>", findClass("Test$", dir).methods)
     }
   }