]> Cypherpunks repositories - gostls13.git/commit
go/types, types2: fix parameter order dependence in type inference
authorRobert Griesemer <gri@golang.org>
Tue, 21 Jun 2022 22:56:16 +0000 (15:56 -0700)
committerRobert Griesemer <gri@golang.org>
Wed, 22 Jun 2022 00:14:46 +0000 (00:14 +0000)
commitb004c739b525d354ea62f5caadd962de4569d96e
tree4efd955dfb82c8c250f201806d78c8c51f09626b
parentf2c7e78592973436a55cdfc4bca2fc3bce526cad
go/types, types2: fix parameter order dependence in type inference

If we have more than two function arguments to a generic function,
we may have arguments with named and unnamed types. If that is the
case, permutate params and args such that the arguments with named
types are first in the list. This way, independent of parameter
ordering, the type inference will produce the same result.

This extra step is not explicitly outlined in the spec yet but we
all agree that (parameter) order independence is an invariant that
we should uphold for type inference. As we move towards less
operational and more descriptive rules for type inference, we will
incorporate this property as well.

The actual fix for this bug existed before 1.18 but was not enabled.
This CL merely enables the fix (switches a flag) and adjusts some
tests.

Fixes #43056.

Change-Id: Ie4e40cf8438dfd82fa94b78068e4f6f6f53f83e6
Reviewed-on: https://go-review.googlesource.com/c/go/+/413459
Reviewed-by: Ian Lance Taylor <iant@google.com>
Reviewed-by: Robert Griesemer <gri@google.com>
src/cmd/compile/internal/types2/infer.go
src/cmd/compile/internal/types2/testdata/examples/functions.go
src/cmd/compile/internal/types2/testdata/fixedbugs/issue43056.go
src/go/types/infer.go
src/go/types/testdata/examples/functions.go
src/go/types/testdata/fixedbugs/issue43056.go