Skip to content
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: internal compiler error: proginfo: unknown instruction 05062 ... BVS 5072 on ppc64le #17005

Closed
laboger opened this issue Sep 6, 2016 · 1 comment
Milestone

Comments

@laboger
Copy link
Contributor

laboger commented Sep 6, 2016

Please answer these questions before submitting your issue. Thanks!

What version of Go are you using (go version)?

go version devel +6db13e0 Tue Sep 6 04:16:42 2016 +0000 linux/ppc64le

What operating system and processor architecture are you using (go env)?

Ubuntu 16.04 ppc64le

What did you do?

If possible, provide a recipe for reproducing the error.
A complete runnable program is good.
A link on play.golang.org is best.

Built go from latest master, tried to build this package:
go get -u github.com/gonum/blas/native

What did you expect to see?

A successful build of the packages

What did you see instead?

src/github.com/gonum/blas/native/level1double.go:351: internal compiler error: proginfo: unknown instruction 05062 (/home/boger/gocode/src/github.com/gonum/blas/native/level1double.go:423) BVS 5072

goroutine 1 [running]:
runtime/debug.Stack(0x0, 0x0, 0x0)
/home/boger/golang/test/go/src/runtime/debug/stack.go:24 +0x6c
cmd/compile/internal/gc.Fatalf(0x6c42c3, 0x20, 0xc420d9b0d8, 0x1, 0x1)
/home/boger/golang/test/go/src/cmd/compile/internal/gc/subr.go:165 +0x1cc
cmd/compile/internal/ppc64.proginfo(0xc4201368f8)
/home/boger/golang/test/go/src/cmd/compile/internal/ppc64/prog.go:150 +0x45c
cmd/compile/internal/gc.newcfg(0xc42012c278, 0xc42012c278, 0xc420eaafe0, 0x2)
/home/boger/golang/test/go/src/cmd/compile/internal/gc/plive.go:403 +0x100
cmd/compile/internal/gc.liveness(0xc420892480, 0xc42012c278, 0xc420b1ef50, 0xc420b1efc0)
/home/boger/golang/test/go/src/cmd/compile/internal/gc/plive.go:1730 +0x164
cmd/compile/internal/gc.genssa(0xc4204064b0, 0xc42012c278, 0xc420b1ef50, 0xc420b1efc0)
/home/boger/golang/test/go/src/cmd/compile/internal/gc/ssa.go:4326 +0xa24
cmd/compile/internal/gc.compile(0xc420892480)
/home/boger/golang/test/go/src/cmd/compile/internal/gc/pgen.go:482 +0x880
cmd/compile/internal/gc.funccompile(0xc420892480)
/home/boger/golang/test/go/src/cmd/compile/internal/gc/dcl.go:1258 +0x1b4
cmd/compile/internal/gc.Main()
/home/boger/golang/test/go/src/cmd/compile/internal/gc/main.go:483 +0x22c0
cmd/compile/internal/ppc64.Main()
/home/boger/golang/test/go/src/cmd/compile/internal/ppc64/galign.go:77 +0x380
main.main()
/home/boger/golang/test/go/src/cmd/compile/main.go:41 +0x20c

Builds successfully on go1.7; appears to be related to SSA.
There are various comments in this area with TODO: in the ppc64 ssa.go file so perhaps there is known work to be done.

The BVS only gets generated if invasmun is true, and where that flag is initialized there is this comment:

// TODO: need to work FP comparisons into block jumps

Also, below the code where the BVS is generated, I see this comment:

            // Pass along what we know so liblink can make use of it.
            // TODO: Once we've fully switched to SSA,
            // make liblink leave our output alone.
@dr2chase dr2chase self-assigned this Sep 6, 2016
@gopherbot
Copy link

CL https://golang.org/cl/28573 mentions this issue.

@quentinmit quentinmit added this to the Go1.8 milestone Sep 6, 2016
@golang golang locked and limited conversation to collaborators Sep 7, 2017
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

No branches or pull requests

4 participants