Move WrappedResult to dotty.util

We can't have WrappedResult in tools itself, since that gives a CyclicReference
error in some situations. Specifically, if tools.WrappedResult is compiled first,
we complete its paackage,
which means we complete tools/package.scala
which means we enter its members
which means we process the WrappedResult export
which means we complete WrappedResult
which gives a CyclicReference

An alternative fix would be to avoid the export and use explicit forwarders instead.

Author: odersky

compiler/src/dotty/tools/dotc/core/Decorators.scala

@@ -11,6 +11,7 @@ import dotty.tools.dotc.transform.MegaPhase
 import ast.tpd._
 import scala.language.implicitConversions
 import printing.Formatting._
+import dotty.util.WrappedResult
 
 /** This object provides useful implicit decorators for types defined elsewhere */
 object Decorators {

compiler/src/dotty/tools/package.scala

@@ -24,10 +24,5 @@ package object tools {
   def unsupported(methodName: String): Nothing =
     throw new UnsupportedOperationException(methodName)
 
-  export WrappedResult.{WrappedResult, result}
-    // Equivalent to:
-    //  type WrappedResult[T] = WrappedResult.WrappedResult[T]
-    //  def result[T] given WrappedResult[T] = WrappedResult.result[T]
-    // The export gave a CyclicReference error in some situations
-    // Investigate if this happens again.
+  export util.WrappedResult.{WrappedResult, result}
 }

compiler/src/dotty/tools/WrappedResult.scala renamed to compiler/src/dotty/util/WrappedResult.scala

@@ -1,4 +1,4 @@
-package dotty.tools
+package dotty.util
 
 object WrappedResult {
   opaque type WrappedResult[T] = T