]> Cypherpunks.ru repositories - gostls13.git/commit
[dev.regabi] cmd/compile: use ir.Ident for imported identifiers
authorMatthew Dempsky <mdempsky@google.com>
Sun, 13 Dec 2020 18:35:20 +0000 (10:35 -0800)
committerMatthew Dempsky <mdempsky@google.com>
Tue, 15 Dec 2020 07:53:10 +0000 (07:53 +0000)
commit4c2d66f642286647b640bced33581e8b1665bfe8
treec7fa15c8c2bdfde48dd618f6545fcd8676aad146
parent305d93ef84aed971145b3aa1bce1f9f389bc90c0
[dev.regabi] cmd/compile: use ir.Ident for imported identifiers

This CL substantially reworks how imported declarations are handled,
and fixes a number of issues with dot imports. In particular:

1. It eliminates the stub ir.Name declarations that are created
upfront during import-declaration processing, allowing this to be
deferred to when the declarations are actually needed. (Eventually,
this can be deferred even further so we never have to create ir.Names
w/ ONONAME, but this CL is already invasive/subtle enough.)

2. During noding, we now use ir.Idents to represent uses of imported
declarations, including of dot-imported declarations.

3. Unused dot imports are now reported after type checking, so that we
can correctly distinguish whether composite literal keys are a simple
identifier (struct literals) or expressions (array/slice/map literals)
and whether it might be a use of a dot-imported declaration.

4. It changes the "redeclared" error messages to report the previous
position information in the same style as other compiler error
messages that reference other source lines.

Passes buildall w/ toolstash -cmp.

Fixes #6428.
Fixes #43164.
Fixes #43167.
Updates #42990.

Change-Id: I40a0a780ec40daf5700fbc3cfeeb7300e1055981
Reviewed-on: https://go-review.googlesource.com/c/go/+/277713
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Go Bot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
Trust: Matthew Dempsky <mdempsky@google.com>
17 files changed:
src/cmd/compile/internal/gc/dcl.go
src/cmd/compile/internal/gc/iimport.go
src/cmd/compile/internal/gc/init.go
src/cmd/compile/internal/gc/main.go
src/cmd/compile/internal/gc/noder.go
src/cmd/compile/internal/gc/subr.go
src/cmd/compile/internal/gc/typecheck.go
src/cmd/compile/internal/ir/name.go
src/cmd/compile/internal/types/sizeof_test.go
src/cmd/compile/internal/types/sym.go
test/fixedbugs/bug462.go
test/fixedbugs/issue20415.go
test/fixedbugs/issue43164.dir/a.go [new file with mode: 0644]
test/fixedbugs/issue43164.dir/b.go [new file with mode: 0644]
test/fixedbugs/issue43164.go [new file with mode: 0644]
test/fixedbugs/issue43167.go [new file with mode: 0644]
test/fixedbugs/issue6428.go [new file with mode: 0644]