Skip to content

Commit

Permalink
go/types, types2: operand.AssignableTo must consider Alias types
Browse files Browse the repository at this point in the history
Fixes regression from Go 1.22.

For #67547.

Change-Id: Id0d07d6b24e1eab6ed1c7476d9d9b82d28aee80a
Reviewed-on: https://go-review.googlesource.com/c/go/+/587161
Reviewed-by: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Reviewed-by: Robert Findley <rfindley@google.com>
Reviewed-by: Robert Griesemer <gri@google.com>
Auto-Submit: Robert Griesemer <gri@google.com>
LUCI-TryBot-Result: Go LUCI <golang-scoped@luci-project-accounts.iam.gserviceaccount.com>
  • Loading branch information
griesemer authored and gopherbot committed May 22, 2024
1 parent 44079f3 commit 8d464e4
Show file tree
Hide file tree
Showing 3 changed files with 30 additions and 5 deletions.
6 changes: 4 additions & 2 deletions src/cmd/compile/internal/types2/operand.go
Original file line number Diff line number Diff line change
Expand Up @@ -260,7 +260,9 @@ func (x *operand) assignableTo(check *Checker, T Type, cause *string) (bool, Cod
return true, 0 // avoid spurious errors
}

V := x.typ
origT := T
V := Unalias(x.typ)
T = Unalias(T)

// x's type is identical to T
if Identical(V, T) {
Expand Down Expand Up @@ -386,7 +388,7 @@ func (x *operand) assignableTo(check *Checker, T Type, cause *string) (bool, Cod
x.typ = V.typ
ok, code = x.assignableTo(check, T, cause)
if !ok {
errorf("cannot assign %s (in %s) to %s", V.typ, Vp, T)
errorf("cannot assign %s (in %s) to %s", V.typ, Vp, origT)
return false
}
return true
Expand Down
6 changes: 4 additions & 2 deletions src/go/types/operand.go

Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.

23 changes: 22 additions & 1 deletion src/internal/types/testdata/fixedbugs/issue67547.go
Original file line number Diff line number Diff line change
Expand Up @@ -53,7 +53,28 @@ func _[P int | string]() {
// preserve target type name A in error messages when using Alias types
// (test are run with and without Alias types enabled, so we need to
// keep both A and int in the error message)
_ = A(p /* ERRORx "cannot convert string .* to type (A|int)" */)
_ = A(p /* ERRORx `cannot convert string \(in P\) to type (A|int)` */)
}

func _[P struct{ x int }]() {
var x struct{ x int }
type A = P
var _ A = x // assignment must be valid
}

func _[P struct{ x int }]() {
type A = P
var x A
var _ struct{ x int } = x // assignment must be valid
}

func _[P []int | struct{}]() {
type A = []int
var a A
var p P
// preserve target type name A in error messages when using Alias types
a = p // ERRORx `cannot assign struct{} \(in P\) to (A|\[\]int)`
_ = a
}

// Test case for go.dev/issue/67540.
Expand Down

0 comments on commit 8d464e4

Please sign in to comment.