From 09b71d563a53766ef2ab34beac6ddaea48c56783 Mon Sep 17 00:00:00 2001 From: Carlos Eduardo Seo Date: Mon, 6 Feb 2017 15:33:43 -0200 Subject: [PATCH] cmd/internal/obj/ppc64, cmd/link/internal/ppc64: Change function alignment to 16 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 Reviewed-by: David Chase Run-TryBot: Lynn Boger --- src/cmd/internal/obj/ppc64/asm9.go | 2 +- src/cmd/link/internal/ppc64/l.go | 2 +- 2 files changed, 2 insertions(+), 2 deletions(-) diff --git a/src/cmd/internal/obj/ppc64/asm9.go b/src/cmd/internal/obj/ppc64/asm9.go index 8bfc667cff..53cdb5f92b 100644 --- a/src/cmd/internal/obj/ppc64/asm9.go +++ b/src/cmd/internal/obj/ppc64/asm9.go @@ -53,7 +53,7 @@ type ctxt9 struct { // Instruction layout. const ( - funcAlign = 8 + funcAlign = 16 ) const ( diff --git a/src/cmd/link/internal/ppc64/l.go b/src/cmd/link/internal/ppc64/l.go index f7ae33dd25..c78535be58 100644 --- a/src/cmd/link/internal/ppc64/l.go +++ b/src/cmd/link/internal/ppc64/l.go @@ -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 */ -- 2.50.0