Commit e9285352 authored by Josh Bleecher Snyder's avatar Josh Bleecher Snyder

cmd/compile: use named fields in nodl

We use a struct to allocate two structs simultaneously.
Because we embed structs rather than using named fields,
the compiler generates forwarding method stubs for the
anonymous type.

In theory, the compiler could detect that these stubs are unnecessary:
The value in question has a very limited scope, the methods are not
called, and there are operations where an interface would need
to be satisfied.

This compiler optimization is unlikely to happen, though;
the ROI is likely to be low.

Instead, just give the fields names. Cuts 64k off the cmd/compile binary.

Change-Id: Id10ec69c23cd2dd33306f4c1bc75724e3c571b56
Reviewed-on: https://go-review.googlesource.com/c/go/+/172579
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
Reviewed-by: default avatarMatthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
parent 9dce58d3
......@@ -306,20 +306,20 @@ func nodl(pos src.XPos, op Op, nleft, nright *Node) *Node {
switch op {
case OCLOSURE, ODCLFUNC:
var x struct {
Node
Func
n Node
f Func
}
n = &x.Node
n.Func = &x.Func
n = &x.n
n.Func = &x.f
case ONAME:
Fatalf("use newname instead")
case OLABEL, OPACK:
var x struct {
Node
Name
n Node
m Name
}
n = &x.Node
n.Name = &x.Name
n = &x.n
n.Name = &x.m
default:
n = new(Node)
}
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment