]> Cypherpunks repositories - gostls13.git/commitdiff
cmd/internal/obj/ppc64, cmd/link/internal/ppc64: Change function alignment to 16
authorCarlos Eduardo Seo <cseo@linux.vnet.ibm.com>
Mon, 6 Feb 2017 17:33:43 +0000 (15:33 -0200)
committerLynn Boger <laboger@linux.vnet.ibm.com>
Fri, 5 May 2017 19:03:05 +0000 (19:03 +0000)
The Power processor manual states that "Branches not from the last instruction
of an aligned quadword and not to the first instruction of an aligned quadword
cause inefficiencies in the IBuffer". This changes the function alignment from 8
to 16 bytes to comply with that.

Fixes #18963

Change-Id: Ibce9bf8302110a86c6ab05948569af9ffdfcf4bb
Reviewed-on: https://go-review.googlesource.com/36390
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: David Chase <drchase@google.com>
Run-TryBot: Lynn Boger <laboger@linux.vnet.ibm.com>

src/cmd/internal/obj/ppc64/asm9.go
src/cmd/link/internal/ppc64/l.go

index 8bfc667cff62df8b078fbaef47818217e749c3a8..53cdb5f92b7ec082200f59b766dcd9d3da034831 100644 (file)
@@ -53,7 +53,7 @@ type ctxt9 struct {
 // Instruction layout.
 
 const (
-       funcAlign = 8
+       funcAlign = 16
 )
 
 const (
index f7ae33dd25abdb5bfb507e0c93709ab0d83d2567..c78535be58fe8f7f189508d788c950eeffb5a7cb 100644 (file)
@@ -64,7 +64,7 @@ package ppc64
 const (
        maxAlign  = 32 // max data alignment
        minAlign  = 1  // min data alignment
-       funcAlign = 8
+       funcAlign = 16
 )
 
 /* Used by ../internal/ld/dwarf.go */