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

x/tools/go/ssa/interp: TestGorootTest times out when GOARCH=386 #11916

Open
mwhudson opened this issue Jul 29, 2015 · 3 comments
Open

x/tools/go/ssa/interp: TestGorootTest times out when GOARCH=386 #11916

mwhudson opened this issue Jul 29, 2015 · 3 comments
Labels
Milestone

Comments

@mwhudson
Copy link
Contributor

@mwhudson mwhudson commented Jul 29, 2015

GOARCH=386 go test ./go/ssa/interp/ -v -run TestGorootTest takes a very long time -- probably it's going to hit 10 minutes and time out but I haven't waited that long yet. It's slow but not so stupidly slow when GOARCH=amd64 -- about 52s.

@davecheney

This comment has been minimized.

Copy link
Contributor

@davecheney davecheney commented Jul 29, 2015

I think this is related to ssa only targeting amd64 at the moment, other arches will fall back to the original code gen, but probably with some overhead.

@josharian can probably give more information, and correct my understanding.

On 29 Jul 2015, at 13:35, Michael Hudson-Doyle notifications@github.com wrote:

GOARCH=386 go test ./go/ssa/interp/ -v -run TestGorootTest takes a very long time -- probably it's going to hit 10 minutes and time out but I haven't waited that long yet. It's slow but not so stupidly slow when GOARCH=amd64 -- about 52s.


Reply to this email directly or view it on GitHub.

@mwhudson

This comment has been minimized.

Copy link
Contributor Author

@mwhudson mwhudson commented Jul 29, 2015

Hm, it seems that it works on arm64 but not armhf, so maybe there is some 64/32 bit thing going on. But the tests pass with GOARCH=386 and amd64 with go 1.4 (haven't tried go1.4 on armhf yet).

It hangs after "Input: encoding/ascii85" fwiw.

@rsc

This comment has been minimized.

Copy link
Contributor

@rsc rsc commented Aug 6, 2015

cc @adonovan

@davecheney, this is Alan Donovan's ssa, not the new ssa in the compiler.

@rsc rsc added this to the Unreleased milestone Aug 6, 2015
@gopherbot gopherbot added the Tools label Sep 12, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
4 participants
You can’t perform that action at this time.