mirror of
https://github.com/golang/go.git
synced 2025-12-08 06:10:04 +00:00
[dev.typeparams] cmd/compile: avoid some redundant type construction
This CL updates noder and typecheck to avoid a couple of instances of redundant evaluation of type expressions: 1. When noding struct fields or parameter tuples, check for syntax.Type reuse between adjacent fields and then reuse the corresponding ir.Node type expression. It would perhaps be even better to avoid re-noding the type expression too, but noder's days are numbered anyway, so I'd rather be minimally invasive here. 2. When importing an empty interface, reuse the same cached empty interface instance that is used for empty interfaces that appear in source. This matches types2's behavior, which uses a single types2.Interface instance for all empty interfaces. These changes are motivated by making it possible to migrate from typecheck to types2 while passing toolstash -cmp. Updates #46208. Change-Id: Ia6458894494464d863181db356f3284630c90ffe Reviewed-on: https://go-review.googlesource.com/c/go/+/320789 Run-TryBot: Matthew Dempsky <mdempsky@google.com> Trust: Matthew Dempsky <mdempsky@google.com> TryBot-Result: Go Bot <gobot@golang.org> Reviewed-by: Robert Griesemer <gri@golang.org>
This commit is contained in:
parent
fd54ae8b0c
commit
95748d1b74
2 changed files with 10 additions and 0 deletions
|
|
@ -625,6 +625,9 @@ func (p *noder) params(params []*syntax.Field, dddOk bool) []*ir.Field {
|
||||||
for i, param := range params {
|
for i, param := range params {
|
||||||
p.setlineno(param)
|
p.setlineno(param)
|
||||||
nodes = append(nodes, p.param(param, dddOk, i+1 == len(params)))
|
nodes = append(nodes, p.param(param, dddOk, i+1 == len(params)))
|
||||||
|
if i > 0 && params[i].Type == params[i-1].Type {
|
||||||
|
nodes[i].Ntype = nodes[i-1].Ntype
|
||||||
|
}
|
||||||
}
|
}
|
||||||
return nodes
|
return nodes
|
||||||
}
|
}
|
||||||
|
|
@ -917,6 +920,9 @@ func (p *noder) structType(expr *syntax.StructType) ir.Node {
|
||||||
} else {
|
} else {
|
||||||
n = ir.NewField(p.pos(field), p.name(field.Name), p.typeExpr(field.Type), nil)
|
n = ir.NewField(p.pos(field), p.name(field.Name), p.typeExpr(field.Type), nil)
|
||||||
}
|
}
|
||||||
|
if i > 0 && expr.FieldList[i].Type == expr.FieldList[i-1].Type {
|
||||||
|
n.Ntype = l[i-1].Ntype
|
||||||
|
}
|
||||||
if i < len(expr.TagList) && expr.TagList[i] != nil {
|
if i < len(expr.TagList) && expr.TagList[i] != nil {
|
||||||
n.Note = constant.StringVal(p.basicLit(expr.TagList[i]))
|
n.Note = constant.StringVal(p.basicLit(expr.TagList[i]))
|
||||||
}
|
}
|
||||||
|
|
|
||||||
|
|
@ -745,6 +745,10 @@ func (r *importReader) typ1() *types.Type {
|
||||||
methods[i] = types.NewField(pos, sym, typ)
|
methods[i] = types.NewField(pos, sym, typ)
|
||||||
}
|
}
|
||||||
|
|
||||||
|
if len(embeddeds)+len(methods) == 0 {
|
||||||
|
return types.Types[types.TINTER]
|
||||||
|
}
|
||||||
|
|
||||||
t := types.NewInterface(r.currPkg, append(embeddeds, methods...))
|
t := types.NewInterface(r.currPkg, append(embeddeds, methods...))
|
||||||
|
|
||||||
// Ensure we expand the interface in the frontend (#25055).
|
// Ensure we expand the interface in the frontend (#25055).
|
||||||
|
|
|
||||||
Loading…
Add table
Add a link
Reference in a new issue