cmd/compile: don't initialize blank struct fields

We already skipped blank field initialization in non-global contexts.
This change makes the global context treatment match.

Fixes #31546

Change-Id: I40acce49b0a9deb351ae0da098f4c114e425ec63
Reviewed-on: https://go-review.googlesource.com/c/go/+/173723
Run-TryBot: Josh Bleecher Snyder <josharian@gmail.com>
Reviewed-by: Matthew Dempsky <mdempsky@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>

diff --git a/src/cmd/compile/internal/gc/sinit.go b/src/cmd/compile/internal/gc/sinit.go
index d755ea35ccde607e7aa6961748ed514c43672550..6666e8bb5eb239dc60d020c7f78d01acdc3f6a9b 100644 (file)
--- a/src/cmd/compile/internal/gc/sinit.go
+++ b/src/cmd/compile/internal/gc/sinit.go
@@ -1278,6 +1278,9 @@ func (s *InitSchedule) initplan(n *Node) {
                        if a.Op != OSTRUCTKEY {
                                Fatalf("initplan structlit")
                        }
+                       if a.Sym.IsBlank() {
+                               continue
+                       }
                        s.addvalue(p, a.Xoffset, a.Left)
                }
 

diff --git a/test/fixedbugs/issue31546.go b/test/fixedbugs/issue31546.go
new file mode 100644 (file)
index 0000000..a459d4d
--- /dev/null
+++ b/test/fixedbugs/issue31546.go
@@ -0,0 +1,20 @@
+// run
+
+// Copyright 2019 The Go Authors. All rights reserved.
+// Use of this source code is governed by a BSD-style
+// license that can be found in the LICENSE file.
+
+package main
+
+import (
+       "reflect"
+)
+
+var x = struct{ a, _, c int }{1, 2, 3}
+
+func main() {
+       if i := reflect.ValueOf(x).Field(1).Int(); i != 0 {
+               println("got", i, "want", 0)
+               panic("fail")
+       }
+}