From: Matthew Dempsky <mdempsky@google.com>
Date: Mon, 23 Apr 2018 17:52:16 +0000 (-0700)
Subject: cmd/compile: remove toolstash workaround in bexport.go
X-Git-Tag: go1.11beta1~721
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=545ef1103712dda49542709d48ce030addecb8e7;p=gostls13.git

cmd/compile: remove toolstash workaround in bexport.go

Change-Id: Ie4facdcab4b35cf7d350c4b8fa06a3c5a0c6caeb
Reviewed-on: https://go-review.googlesource.com/108875
Run-TryBot: Matthew Dempsky <mdempsky@google.com>
Reviewed-by: Brad Fitzpatrick <bradfitz@golang.org>
TryBot-Result: Gobot Gobot <gobot@golang.org>
---

diff --git a/src/cmd/compile/internal/gc/bexport.go b/src/cmd/compile/internal/gc/bexport.go
index 6644827635..d71c069cfc 100644
--- a/src/cmd/compile/internal/gc/bexport.go
+++ b/src/cmd/compile/internal/gc/bexport.go
@@ -1002,13 +1002,7 @@ func parName(f *types.Field, numbered bool) string {
 		return ""
 	}
 
-	// The "s != f.Sym" check here is unnecessary and causes blank
-	// input/receiver parameters to receive vargen numbers
-	// below. However, this is consistent with the logic it
-	// replaces, so we keep it for now to appease toolstash-check.
-	//
-	// TODO(mdempsky): Simplify to just "if s.Name == "_"".
-	if s != f.Sym && s.Name == "_" {
+	if s.Name == "_" {
 		return "_"
 	}