]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile/internal/noder: stop creating TUNION types
authorMatthew Dempsky <mdempsky@google.com>
Fri, 2 Dec 2022 01:24:23 +0000 (17:24 -0800)
committerMatthew Dempsky <mdempsky@google.com>
Thu, 26 Jan 2023 21:43:08 +0000 (21:43 +0000)
In the types1 universe under the unified frontend, we never need to
worry about type parameter constraints, so we only see pure
interfaces. However, we might still see interfaces that contain union
types, because of interfaces like "interface{ any | int }" (equivalent
to just "any").

We can handle these without needing to actually represent type unions
within types1 by simply mapping any union to "any".

Updates #57410.

Change-Id: I5e4efcf0339edbb01f4035c54fb6fb1f9ddc0c65
Reviewed-on: https://go-review.googlesource.com/c/go/+/458619
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
Reviewed-by: Keith Randall <khr@google.com>
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/noder/reader.go
test/typeparam/issue52124.go

index bdec467f90ab72286f95c624aa6346b42c811aa4..ac6d2fa9319263fcf7ea2cee509205de25268bf5 100644 (file)
@@ -519,13 +519,33 @@ func (r *reader) doTyp() *types.Type {
 }
 
 func (r *reader) unionType() *types.Type {
-       terms := make([]*types.Type, r.Len())
-       tildes := make([]bool, len(terms))
-       for i := range terms {
-               tildes[i] = r.Bool()
-               terms[i] = r.typ()
+       // In the types1 universe, we only need to handle value types.
+       // Impure interfaces (i.e., interfaces with non-trivial type sets
+       // like "int | string") can only appear as type parameter bounds,
+       // and this is enforced by the types2 type checker.
+       //
+       // However, type unions can still appear in pure interfaces if the
+       // type union is equivalent to "any". E.g., typeparam/issue52124.go
+       // declares variables with the type "interface { any | int }".
+       //
+       // To avoid needing to represent type unions in types1 (since we
+       // don't have any uses for that today anyway), we simply fold them
+       // to "any". As a consistency check, we still read the union terms
+       // to make sure this substitution is safe.
+
+       pure := false
+       for i, n := 0, r.Len(); i < n; i++ {
+               _ = r.Bool() // tilde
+               term := r.typ()
+               if term.IsEmptyInterface() {
+                       pure = true
+               }
+       }
+       if !pure {
+               base.Fatalf("impure type set used in value type")
        }
-       return types.NewUnion(terms, tildes)
+
+       return types.Types[types.TINTER]
 }
 
 func (r *reader) interfaceType() *types.Type {
index a113fc74441f118c512b82f37dd61d4720816a76..07cba479821ee2c6e7afd6d608e02ace0fcaf3db 100644 (file)
@@ -6,7 +6,9 @@
 
 package p
 
-type I interface{ any | int }
+type Any any
+
+type I interface{ Any | int }
 
 var (
        X I = 42