-
Notifications
You must be signed in to change notification settings - Fork 17.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
cmd/compile: misuse of REGTMP in optimizer on ppc64 #12597
Labels
Milestone
Comments
On linux/ppc64, TestInterpClipCommute and TestRectDstMask also fails. It's probably due to missing narrowing masks. |
Can this be added to the go 1.5.2 milestone please? |
Done (without actually looking at the issue). |
It's a ppc64 codegen bug (optimizer keeps a live variable in REGTMP).
|
CL https://golang.org/cl/16981 mentions this issue. |
aclements
pushed a commit
that referenced
this issue
Nov 17, 2015
…pc64 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>
mwhudson
added a commit
to mwhudson/go
that referenced
this issue
Jan 4, 2016
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 golang#12597 Change-Id: I29c7655e32abd22f3c21d88427b73e4fca055233
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
Probably a codegen bug.
The text was updated successfully, but these errors were encountered: