diff options
| author | Michael Hudson-Doyle <michael.hudson@canonical.com> | 2015-10-05 16:07:59 +1300 |
|---|---|---|
| committer | Austin Clements <austin@google.com> | 2015-11-17 02:23:39 +0000 |
| commit | 8b3f554d869e1d6e7a0bd1574135402cfc6dc1a3 (patch) | |
| tree | 60c9a9e3d962067c522bee44925fa3a448dc4085 | |
| parent | 244294f821c9256394d29218ee1f17ab4313551f (diff) | |
| download | go-8b3f554d869e1d6e7a0bd1574135402cfc6dc1a3.tar.gz go-8b3f554d869e1d6e7a0bd1574135402cfc6dc1a3.zip | |
[release-branch.go1.5] cmd/compile: do not let regopt use REGTMP on ppc64
ppc64 codegen assumes that it is OK to stomp on r31 at any time, but it is not
excluded from the set of registers that regopt is allowed to use.
Fixes #12597
Change-Id: I29c7655e32abd22f3c21d88427b73e4fca055233
Reviewed-on: https://go-review.googlesource.com/15245
Reviewed-by: Minux Ma <minux@golang.org>
Reviewed-on: https://go-review.googlesource.com/16981
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
| -rw-r--r-- | src/cmd/compile/internal/ppc64/reg.go | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/cmd/compile/internal/ppc64/reg.go b/src/cmd/compile/internal/ppc64/reg.go index fa1cb71975..a3018362f6 100644 --- a/src/cmd/compile/internal/ppc64/reg.go +++ b/src/cmd/compile/internal/ppc64/reg.go @@ -111,7 +111,7 @@ func regnames(n *int) []string { func excludedregs() uint64 { // Exclude registers with fixed functions - regbits := uint64(1<<0 | RtoB(ppc64.REGSP) | RtoB(ppc64.REGG) | RtoB(ppc64.REGTLS)) + regbits := uint64(1<<0 | RtoB(ppc64.REGSP) | RtoB(ppc64.REGG) | RtoB(ppc64.REGTLS) | RtoB(ppc64.REGTMP)) // Also exclude floating point registers with fixed constants regbits |= RtoB(ppc64.REG_F27) | RtoB(ppc64.REG_F28) | RtoB(ppc64.REG_F29) | RtoB(ppc64.REG_F30) | RtoB(ppc64.REG_F31) |