]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/compile: remove toolstash workaround in bexport.go
authorMatthew Dempsky <mdempsky@google.com>
Mon, 23 Apr 2018 17:52:16 +0000 (10:52 -0700)
committerMatthew Dempsky <mdempsky@google.com>
Mon, 23 Apr 2018 18:06:05 +0000 (18:06 +0000)
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>

src/cmd/compile/internal/gc/bexport.go

index 66448276358ac866d67bdd53206b0ebcf5fd3fcf..d71c069cfcf2afcdf17f8bc0a05b7a587af91e43 100644 (file)
@@ -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 "_"
        }