Make type of type hole available quoted patterns

In general, instances of `scala.quoted.Type` are and should not be used explicitly in quotes.

```diff
+ def f[T](x: T)(using Type[T])(using QuoteContext) =
+   '{ val y: T = $x; ... } // good code

- def f[T](x: T)(using t: Type[T])(using QuoteContext) =
-   '{ val y: t.T = $x; ... } // bad code

- def f[T](x: T)(using t: Type[T])(using QuoteContext) =
-   '{ val y: $t = $x; ... } // bad code (syntax will be removed)
```

The only exception is on types extracted using quoted patterns as these hide tier type inside an instance of `Type[T]` which must be accessed using it's path.
```diff
??? match
  case '{ $x: $t } =>
-   '{ val y: t.T = $x; ... } // bad code
-   '{ val y: $t = $x; ... } // bad code (syntax will be removed)
```

Here `t` is provided as a `given` as if we refer to its type we must be able to summon it.

The change is to provide the name of the type that is extracted rather than the `given Type[T]`.
`Type[T]` can be summoned were needed as in all other uses of abstract types in quotes.

```diff
??? match
  case '{ $x: $T } =>
+   '{ val y: T = $x; ... } // good code
-   val t: Type[T] = summon[Type[T]]
-   '{ val y: t.T = $x; ... } // bad code
-   '{ val y: $t = $x; ... } // bad code (syntax will be removed)
```

The pattern will make `T` and a `given Type[T]` available on the right-hand side of the pattern.
The `T` can be used the same way it could be used in the implementation of a method with signature:
```scala
def f[T](x: T)(using Type[T])(using QuoteContext) =
  '{ val y: T = $x; ... }
```

```scala
case '{ $x: $T } =>
  '{ val y: T = $x; ... }
```

It is also possible to use lower cases for the type splices `case '{ $x: $t } => '{ val y: t = $x; ... }`.
This is similar to the use of type variables in normal patterns `case Some[t](x) =>`.
It is preferable to use upper cases as it makes it clearer that this is a type and renders better on the uses in the right-hand side of the pattern.

Author: nicolasstucki

community-build/community-projects/scalatest

@@ -320,6 +320,7 @@ object NameKinds {
   val PatMatStdBinderName: UniqueNameKind    = new UniqueNameKind("x")
   val PatMatAltsName: UniqueNameKind         = new UniqueNameKind("matchAlts")
   val PatMatResultName: UniqueNameKind       = new UniqueNameKind("matchResult")
+  val PatMatQuoteTypeEv: UniqueNameKind      = new UniqueNameKind("tpev$")
 
   val LocalOptInlineLocalObj: UniqueNameKind = new UniqueNameKind("ilo")
 

community-build/community-projects/shapeless

@@ -80,15 +80,16 @@ object NameOps {
       case _ => false
 
     /** Is name a variable name? */
-    def isVariableName: Boolean = testSimple { n =>
-      n.length > 0 && {
-        val first = n.head
-        (((first.isLower && first.isLetter) || first == '_')
-          && (n != false_)
-          && (n != true_)
-          && (n != null_))
-      }
-    }
+    def isVariableName: Boolean =
+      testSimple { n =>
+        n.length > 0 && {
+          val first = n.head
+          (((first.isLower && first.isLetter) || first == '_')
+            && (n != false_)
+            && (n != true_)
+            && (n != null_))
+        }
+      } || name.is(PatMatQuoteTypeEv)
 
     def isOpAssignmentName: Boolean = name match {
       case raw.NE | raw.LE | raw.GE | EMPTY =>

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

@@ -9,7 +9,7 @@ import dotty.tools.dotc.core.Constants._
 import dotty.tools.dotc.core.Contexts._
 import dotty.tools.dotc.core.Decorators._
 import dotty.tools.dotc.core.Flags._
-import dotty.tools.dotc.core.NameKinds.UniqueName
+import dotty.tools.dotc.core.NameKinds.{UniqueName, PatMatQuoteTypeEv}
 import dotty.tools.dotc.core.Names._
 import dotty.tools.dotc.core.StagingContext._
 import dotty.tools.dotc.core.StdNames._
@@ -155,19 +155,21 @@ trait QuotesAndSplices {
     if ctx.mode.is(Mode.QuotedPattern) && level == 1 then
       def spliceOwner(ctx: Context): Symbol =
       if (ctx.mode.is(Mode.QuotedPattern)) spliceOwner(ctx.outer) else ctx.owner
-      val name = tree.expr match {
-        case Ident(name) => ("$" + name).toTypeName
+      val (name, expr) = tree.expr match {
+        case Ident(name) =>
+          val nameOfSyntheticGiven = PatMatQuoteTypeEv.fresh()
+          (name.toTypeName, untpd.cpy.Ident(tree.expr)(nameOfSyntheticGiven))
         case expr =>
           report.error("expected a name binding", expr.srcPos)
-          "$error".toTypeName
+          ("$error".toTypeName, expr)
       }
 
       val typeSymInfo = pt match
         case pt: TypeBounds => pt
         case _ => TypeBounds.empty
       val typeSym = newSymbol(spliceOwner(ctx), name, EmptyFlags, typeSymInfo, NoSymbol, tree.expr.span)
       typeSym.addAnnotation(Annotation(New(ref(defn.InternalQuotedPatterns_patternTypeAnnot.typeRef)).withSpan(tree.expr.span)))
-      val pat = typedPattern(tree.expr, defn.QuotedTypeClass.typeRef.appliedTo(typeSym.typeRef))(
+      val pat = typedPattern(expr, defn.QuotedTypeClass.typeRef.appliedTo(typeSym.typeRef))(
           using spliceContext.retractMode(Mode.QuotedPattern).withOwner(spliceOwner(ctx)))
       pat.select(tpnme.spliceType)
     else
@@ -211,7 +213,7 @@ trait QuotesAndSplices {
     def getBinding(sym: Symbol): Bind =
       typeBindings.getOrElseUpdate(sym, {
         val bindingBounds = sym.info
-        val bsym = newPatternBoundSymbol(sym.name.toTypeName, bindingBounds, quoted.span)
+        val bsym = newPatternBoundSymbol(sym.name.toString.stripPrefix("$").toTypeName, bindingBounds, quoted.span)
         Bind(bsym, untpd.Ident(nme.WILDCARD).withType(bindingBounds)).withSpan(quoted.span)
       })
 
@@ -260,13 +262,14 @@ trait QuotesAndSplices {
           val sym = tree.tpe.dealias.typeSymbol
           if sym.exists then
             val tdef = TypeDef(sym.asType).withSpan(sym.span)
-            freshTypeBindingsBuff += transformTypeBindingTypeDef(tdef, freshTypePatBuf)
+            val nameOfSyntheticGiven = pat.symbol.name.toTermName
+            freshTypeBindingsBuff += transformTypeBindingTypeDef(nameOfSyntheticGiven, tdef, freshTypePatBuf)
             TypeTree(tree.tpe.dealias).withSpan(tree.span)
           else
             tree
         case tdef: TypeDef  =>
           if tdef.symbol.hasAnnotation(defn.InternalQuotedPatterns_patternTypeAnnot) then
-            transformTypeBindingTypeDef(tdef, typePatBuf)
+            transformTypeBindingTypeDef(PatMatQuoteTypeEv.fresh(), tdef, typePatBuf)
           else if tdef.symbol.isClass then
             val kind = if tdef.symbol.is(Module) then "objects" else "classes"
             report.error("Implementation restriction: cannot match " + kind, tree.srcPos)
@@ -302,13 +305,12 @@ trait QuotesAndSplices {
           super.transform(tree)
       }
 
-      private def transformTypeBindingTypeDef(tdef: TypeDef, buff: mutable.Builder[Tree, List[Tree]])(using Context): Tree = {
+      private def transformTypeBindingTypeDef(nameOfSyntheticGiven: TermName, tdef: TypeDef, buff: mutable.Builder[Tree, List[Tree]])(using Context): Tree = {
         if (variance == -1)
           tdef.symbol.addAnnotation(Annotation(New(ref(defn.InternalQuotedPatterns_fromAboveAnnot.typeRef)).withSpan(tdef.span)))
         val bindingType = getBinding(tdef.symbol).symbol.typeRef
         val bindingTypeTpe = AppliedType(defn.QuotedTypeClass.typeRef, bindingType :: Nil)
-        val bindName = tdef.name.toString.stripPrefix("$").toTermName
-        val sym = newPatternBoundSymbol(bindName, bindingTypeTpe, tdef.span, flags = ImplicitTerm)(using ctx0)
+        val sym = newPatternBoundSymbol(nameOfSyntheticGiven, bindingTypeTpe, tdef.span, flags = ImplicitTerm)(using ctx0)
         buff += Bind(sym, untpd.Ident(nme.WILDCARD).withType(bindingTypeTpe)).withSpan(tdef.span)
         super.transform(tdef)
       }

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

@@ -1,10 +1,10 @@
 import scala.quoted._
 object Test {
-  def test(x: quoted.Expr[Int])(using QuoteContext) = x match {
-    case '{ type $t; poly[$t]($x); 4 } => ??? // error: duplicate pattern variable: $t
-    case '{ type `$t`; poly[`$t`]($x); 4 } =>
-      val tt: quoted.Type[_] = t // error
-      ???
+  def test(x: quoted.Expr[Int])(using QuoteContext): Unit = x match {
+    case '{ type $T; 4 } => Type[T]
+    case '{ type $T; poly[$T]($x); 4 } => // error: duplicate pattern variable: T
+    case '{ type `$T`; poly[`$T`]($x); 4 } =>
+      Type[T] // error
     case _ =>
   }
 

compiler/src/dotty/tools/dotc/typer/QuotesAndSplices.scala

@@ -6,9 +6,9 @@ inline def mcr(x: => Any): Any = ${mcrImpl('x)}
 
 def mcrImpl(body: Expr[Any])(using ctx: QuoteContext): Expr[Any] =
   body match
-    case '{$x: $t} =>
+    case '{$x: $T} =>
       '{
-        val tmp: $t = $x
+        val tmp: T = $x
         println(tmp)
         tmp
       }

tests/neg-macros/quotedPatterns-5.scala

@@ -1,7 +1,7 @@
 import scala.quoted._
 class Foo {
   def f[T2](t: Type[T2])(using QuoteContext) = t match {
-    case '[ *:[Int, $t] ] =>
-      '[ *:[Int, $t] ]
+    case '[ *:[Int, $T] ] =>
+      '[ *:[Int, T] ]
   }
 }

tests/pos-macros/i6997c.scala

@@ -2,6 +2,6 @@ import scala.quoted._
 class Foo {
   def f[T2: Type](e: Expr[T2])(using QuoteContext) = e match {
     case '{ $x: *:[Int, $t] } =>
-      '[ *:[Int, $t] ]
+      '[ *:[Int, t] ]
   }
 }

tests/pos-macros/i7264.scala

@@ -1,9 +1,9 @@
 import scala.quoted._
 class Foo {
   def f[T2: Type](e: Expr[T2])(using QuoteContext) = e match {
-    case '{ $x: $t0 } =>
-      t0 match
-        case '[ *:[Int, $t] ] =>
-          '[ *:[Int, $t] ]
+    case '{ $x: $T0 } =>
+      Type[T0] match
+        case '[ *:[Int, $T] ] =>
+          '[ *:[Int, T] ]
   }
 }

tests/pos-macros/i7264b.scala

@@ -4,15 +4,15 @@ object Macro {
 
   trait AddInt[A <: Int, B <: Int] { type Out <: Int }
 
-  transparent inline def ff[A <: Int, B <: Int](): AddInt[A, B] = ${ impl('[A], '[B]) }
+  transparent inline def ff[A <: Int, B <: Int](): AddInt[A, B] = ${ impl[A, B] }
 
-  def impl[A <: Int : Type, B <: Int : Type](a: Type[A], b: Type[B])(using qctx: QuoteContext) : Expr[AddInt[A, B]] = {
+  def impl[A <: Int : Type, B <: Int : Type](using qctx: QuoteContext) : Expr[AddInt[A, B]] = {
     import qctx.reflect._
 
-    val ConstantType(Constant.Int(v1)) = a.unseal.tpe
-    val ConstantType(Constant.Int(v2)) = b.unseal.tpe
+    val ConstantType(Constant.Int(v1)) = Type.of[A]
+    val ConstantType(Constant.Int(v2)) = Type.of[B]
 
     Literal(Constant.Int(v1 + v2)).tpe.seal match
-      case '[$t] => '{ null: AddInt[$a, $b] { type Out = $t } }
+      case '[$T] => '{ null: AddInt[A, B] { type Out = T } }
   }
 }

tests/pos-macros/i7264c.scala

@@ -59,7 +59,7 @@ class CheckedTransformation(transform: PartialFunction[Expr[Any], Expr[Any]]) ex
   def apply[T: Type](e: Expr[T])(using QuoteContext): Expr[T] = {
     transform.applyOrElse(e, identity) match {
       case '{ $e2: T } => e2
-      case '{ $e2: $t } =>
+      case '{ $e2: $T } =>
         throw new Exception(
           s"""Transformed
             |${e.show}
@@ -69,7 +69,7 @@ class CheckedTransformation(transform: PartialFunction[Expr[Any], Expr[Any]]) ex
             |Expected type to be
             |${summon[Type[T]].show}
             |but was
-            |${t.show}
+            |${Type[T].show}
           """.stripMargin)
     }
   }

tests/pos-macros/tasty-constant-type/Macro_1.scala

@@ -49,7 +49,7 @@ private class Rewriter(preTransform: Expr[Any] => Expr[Any], postTransform: Expr
            |Expected type to be
            |${summon[Type[T]].show}
            |but was
-           |${t.show}
+           |${Type[t].show}
          """.stripMargin)
     }
   }

tests/run-macros/flops-rewrite-3/Macro_1.scala

@@ -7,6 +7,6 @@ object Macros {
 
   def macroImpl[T]()(using qctx: QuoteContext): Expr[String] = {
     Expr.summon[Mirror.Of[Some[Int]]] match
-      case Some('{ $_ : $t }) => Expr(t.show)
+      case Some('{ $_ : $T }) => Expr(Type[T].show)
   }
 }

tests/run-macros/flops-rewrite/Macro_1.scala

@@ -4,9 +4,9 @@ import scala.quoted._
 
 object Macro1 {
 
-  def mirrorFields[T](t: Type[T])(using qctx: QuoteContext): List[String] =
-    t match {
-      case '[$field *: $fields] => field.show :: mirrorFields(fields)
+  def mirrorFields[T: Type](using qctx: QuoteContext): List[String] =
+    Type[T] match {
+      case '[$Field *: $Fields] => Type[Field].show :: mirrorFields[Fields]
       case '[EmptyTuple] => Nil
     }
 
@@ -22,8 +22,8 @@ object Macro1 {
     val mirrorTpe = '[Mirror.Of[T]]
 
     Expr.summon(using mirrorTpe).get match {
-      case '{ $m: Mirror.ProductOf[T]{ type MirroredElemLabels = $t } } => {
-        Expr(mirrorFields(t))
+      case '{ $m: Mirror.ProductOf[T]{ type MirroredElemLabels = $Elems } } => {
+        Expr(mirrorFields[Elems])
       }
     }
   }

tests/run-macros/i7987/Macros_1.scala

@@ -4,9 +4,9 @@ import scala.quoted._
 
 object Macro2 {
 
-  def mirrorFields[T](t: Type[T])(using qctx: QuoteContext): List[String] =
+  def mirrorFields[T](using t: Type[T])(using qctx: QuoteContext): List[String] =
     t match {
-      case '[$field *: $fields] => field.show.substring(1, field.show.length-1) :: mirrorFields(fields)
+      case '[$Field *: $Fields] => Type[Field].show.substring(1, Type[Field].show.length-1) :: mirrorFields[Fields]
       case '[EmptyTuple] => Nil
     }
 
@@ -24,8 +24,8 @@ object Macro2 {
       import qctx.reflect._
 
       val fields = ev match {
-        case '{ $m: Mirror.ProductOf[T] { type MirroredElemLabels = $t } } =>
-          mirrorFields(t)
+        case '{ $m: Mirror.ProductOf[T] { type MirroredElemLabels = $Labels } } =>
+          mirrorFields[Labels]
       }
 
       val body: Expr[T] => Expr[String] = elem =>

tests/run-macros/i8007/Macro_1.scala

@@ -25,10 +25,10 @@ object Eq {
       def eqv(x: T, y: T): Boolean = body(x, y)
     }
 
-  def summonAll[T](t: Type[T])(using qctx: QuoteContext): List[Expr[Eq[_]]] = t match {
-    case '[String *: $tpes] => '{ summon[Eq[String]] }  :: summonAll(tpes)
-    case '[Int *: $tpes]    => '{ summon[Eq[Int]] }     :: summonAll(tpes)
-    case '[$tpe *: $tpes]   => derived(using tpe, qctx) :: summonAll(tpes)
+  def summonAll[T](using t: Type[T])(using qctx: QuoteContext): List[Expr[Eq[_]]] = t match {
+    case '[String *: $Tpes] => '{ summon[Eq[String]] }  :: summonAll[Tpes]
+    case '[Int *: $Tpes]    => '{ summon[Eq[Int]] }     :: summonAll[Tpes]
+    case '[$Tpe *: $Tpes]   => derived[Tpe] :: summonAll[Tpes]
     case '[EmptyTuple] => Nil
   }
 
@@ -38,8 +38,8 @@ object Eq {
     val ev: Expr[Mirror.Of[T]] = Expr.summon(using '[Mirror.Of[T]]).get
 
     ev match {
-      case '{ $m: Mirror.ProductOf[T] { type MirroredElemTypes = $elementTypes }} =>
-        val elemInstances = summonAll(elementTypes)
+      case '{ $m: Mirror.ProductOf[T] { type MirroredElemTypes = $ElementTypes }} =>
+        val elemInstances = summonAll[ElementTypes]
         val eqProductBody: (Expr[T], Expr[T]) => Expr[Boolean] = (x, y) => {
           elemInstances.zipWithIndex.foldLeft(Expr(true: Boolean)) {
             case (acc, (elem, index)) =>
@@ -53,8 +53,8 @@ object Eq {
           eqProduct((x: T, y: T) => ${eqProductBody('x, 'y)})
         }
 
-      case '{ $m: Mirror.SumOf[T] { type MirroredElemTypes = $elementTypes }} =>
-        val elemInstances = summonAll(elementTypes)
+      case '{ $m: Mirror.SumOf[T] { type MirroredElemTypes = $ElementTypes }} =>
+        val elemInstances = summonAll[ElementTypes]
         val eqSumBody: (Expr[T], Expr[T]) => Expr[Boolean] = (x, y) => {
           val ordx = '{ $m.ordinal($x) }
           val ordy = '{ $m.ordinal($y) }

tests/run-macros/i8007/Macro_2.scala

@@ -20,7 +20,7 @@ object Macros {
     object FromEnv {
       def unapply[T](e: Expr[Any])(using env: Env): Option[Expr[R[T]]] =
         e match
-          case '{envVar[$t](${Const(id)})} =>
+          case '{envVar[$_](${Const(id)})} =>
             env.get(id).asInstanceOf[Option[Expr[R[T]]]] // We can only add binds that have the same type as the refs
           case _ =>
             None
@@ -39,11 +39,11 @@ object Macros {
       case '{ ($x: Int) <= ($y: Int) } =>
         '{ $sym.leq(${lift(x)}, ${lift(y)}).asInstanceOf[R[T]] }
 
-      case '{ ${f}($arg: $t): $u } =>
-        '{ $sym.app[$t, $u](${lift(f)}, ${lift(arg)}).asInstanceOf[R[T]] }
+      case '{ ${f}($arg: $A): $B } =>
+        '{ $sym.app[A, B](${lift(f)}, ${lift(arg)}).asInstanceOf[R[T]] }
 
-      case '{ (if ($cond) $thenp else $elsep): $t } =>
-        '{ $sym.ifThenElse[$t](${lift(cond)}, ${lift(thenp)}, ${lift(elsep)}) }.asInstanceOf[Expr[R[T]]]
+      case '{ (if ($cond) $thenp else $elsep): $A } =>
+        '{ $sym.ifThenElse[A](${lift(cond)}, ${lift(thenp)}, ${lift(elsep)}) }.asInstanceOf[Expr[R[T]]]
 
       case '{ (x0: Int) => $bodyFn(x0): Any } =>
         val (i, nEnvVar) = freshEnvVar[Int]()
@@ -60,8 +60,8 @@ object Macros {
         val body2 = UnsafeExpr.open(bodyFn) { (body1, close) => close(body1)(nEnvVar) }
         '{ $sym.lam((x: R[Int => Int]) => ${given Env = envWith(i, 'x)(using env); lift(body2)}).asInstanceOf[R[T]] }
 
-      case '{ Symantics.fix[$t, $u]($f) } =>
-        '{ $sym.fix[$t, $u]((x: R[$t => $u]) => $sym.app(${lift(f)}, x)).asInstanceOf[R[T]] }
+      case '{ Symantics.fix[$A, $B]($f) } =>
+        '{ $sym.fix[A, B]((x: R[A => B]) => $sym.app(${lift(f)}, x)).asInstanceOf[R[T]] }
 
       case FromEnv(expr) => expr.asInstanceOf[Expr[R[T]]]
 

tests/run-macros/i8007/Macro_3.scala

@@ -7,8 +7,8 @@ object Macros {
 
   private def impl(x: Expr[Unit])(using QuoteContext): Expr[Unit] = {
     x match {
-      case '{ DSL.f[$t]($x) } => '{ DSL.g[$t]($x) }
-      case '{ DSL.g[$t]($x) } => '{ DSL.f[$t]($x) }
+      case '{ DSL.f[$T]($x) } => '{ DSL.g[T]($x) }
+      case '{ DSL.g[$T]($x) } => '{ DSL.f[T]($x) }
       case _ => x
     }
   }

tests/run-macros/quote-matcher-symantics-3/quoted_1.scala

@@ -2,17 +2,17 @@ import scala.quoted._
 
 object Macros {
 
-  inline def lift[A]: String = ${ matchesExpr('[A]) }
+  inline def lift[A]: String = ${ matchesExpr[A] }
 
-  private def matchesExpr(tp: Type[_])(using QuoteContext): Expr[String] = {
-    def lift(tp: Type[_]): String = tp match {
+  private def matchesExpr[A](using tp: Type[A])(using QuoteContext): Expr[String] = {
+    def lift[T](using tp: Type[T]): String = tp match {
       case '[Int] => "%Int%"
-      case '[List[$t]] => s"%List[${lift(t)}]%"
-      case '[Option[$t]] => s"%Option[${lift(t)}]%"
-      case '[Function1[$t, $u]] => s"%${lift(t)} => ${lift(u)}%"
+      case '[List[$T]] => s"%List[${lift[T]}]%"
+      case '[Option[$T]] => s"%Option[${lift[T]}]%"
+      case '[Function1[$T, $U]] => s"%${lift[T]} => ${lift[U]}%"
       case _ => tp.show
     }
-    Expr(lift(tp))
+    Expr(lift[A])
   }
 
 }