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 8ad3084410cf17a70d6b08a704cf3f46f85579e0..c6866e65e79fb2b3a1c13d8cca8c7bdc5537ea92 100644 (file)
--- 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 ea66baa007da1802a344dc730d75870f3ca559bf..a63a0f0f7756858eea636757746bcf6d0ff4c9eb 100644 (file)
--- 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