]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: fix ICE when checking implicit dot for method call
authorCuong Manh Le <cuong.manhle.vn@gmail.com>
Tue, 9 Aug 2022 12:30:47 +0000 (19:30 +0700)
committerGopher Robot <gobot@golang.org>
Mon, 15 Aug 2022 17:49:19 +0000 (17:49 +0000)
commite99f285d52627aa415f9c1766cbffe375b2f67d9
treec015984cce5096079cbe3051215bf00c05b09c9a
parent1f833e4a1bfd77c2dfa91305f90ade9f9ceb66de
cmd/compile: fix ICE when checking implicit dot for method call

CL 414836 limited the check for implicit dot for method call enabled by
a type bound. However, the checking condition for ODOTMETH only is not
right. For example, for promoted method, we have a OXDOT node instead,
and we still have to check for implicit dot in this case.

However, if the base type and embedded types have the same method name,
e.g in issue #53419, typecheck.AddImplicitDots will be confused and
result in an ambigus selector.

To fix this, we ensure methods for the base type are computed, then only
do the implicit dot check if we can find a matched method.

Fixes #54348

Change-Id: Iefe84ff330830afe35c5daffd499824db108da23
Reviewed-on: https://go-review.googlesource.com/c/go/+/422274
TryBot-Result: Gopher Robot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@google.com>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
Run-TryBot: Cuong Manh Le <cuong.manhle.vn@gmail.com>
Auto-Submit: Keith Randall <khr@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
src/cmd/compile/internal/noder/stencil.go
test/fixedbugs/issue54348.go [new file with mode: 0644]