From: Lynn Boger <laboger@linux.vnet.ibm.com>
Date: Wed, 1 May 2019 14:55:27 +0000 (-0400)
Subject: cmd/compile/internal/ppc64: improve naming for ginsnop2
X-Git-Tag: go1.13beta1~456
X-Git-Url: http://www.git.cypherpunks.su/?a=commitdiff_plain;h=b098c0f467e5ce70b936381c439a0cbafc3316d3;p=gostls13.git

cmd/compile/internal/ppc64: improve naming for ginsnop2

This is a follow up from a review comment at the end of the last
Go release, to provide a more meaningful name for ginsnop2.

Updates #30475

Change-Id: Ice9efd763bf2204a9e8c55ae230d3e8a80210108
Reviewed-on: https://go-review.googlesource.com/c/go/+/174757
Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Keith Randall <khr@golang.org>
---

diff --git a/src/cmd/compile/internal/ppc64/galign.go b/src/cmd/compile/internal/ppc64/galign.go
index 8ad3084410..c6866e65e7 100644
--- a/src/cmd/compile/internal/ppc64/galign.go
+++ b/src/cmd/compile/internal/ppc64/galign.go
@@ -21,7 +21,7 @@ func Init(arch *gc.Arch) {
 	arch.ZeroRange = zerorange
 	arch.ZeroAuto = zeroAuto
 	arch.Ginsnop = ginsnop
-	arch.Ginsnopdefer = ginsnop2
+	arch.Ginsnopdefer = ginsnopdefer
 
 	arch.SSAMarkMoves = ssaMarkMoves
 	arch.SSAGenValue = ssaGenValue
diff --git a/src/cmd/compile/internal/ppc64/ggen.go b/src/cmd/compile/internal/ppc64/ggen.go
index ea66baa007..a63a0f0f77 100644
--- a/src/cmd/compile/internal/ppc64/ggen.go
+++ b/src/cmd/compile/internal/ppc64/ggen.go
@@ -67,8 +67,9 @@ func ginsnop(pp *gc.Progs) *obj.Prog {
 	return p
 }
 
-func ginsnop2(pp *gc.Progs) *obj.Prog {
-	// PPC64 is unusual because TWO nops are required
+func ginsnopdefer(pp *gc.Progs) *obj.Prog {
+	// On PPC64 two nops are required in the defer case.
+	//
 	// (see gc/cgen.go, gc/plive.go -- copy of comment below)
 	//
 	// On ppc64, when compiling Go into position