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.

Sign up for GitHub

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

Jump to bottom

cmd/compile: internal compiler error: out of fixed registers on 386 #11917

Closed
alexbrainman opened this issue Jul 29, 2015 · 5 comments
Closed

cmd/compile: internal compiler error: out of fixed registers on 386 #11917

alexbrainman opened this issue Jul 29, 2015 · 5 comments
Labels
FrozenDueToAge
Milestone
Go1.5

Comments

@alexbrainman
Copy link
Member

Reported by windows-386 builder building "exp" sub-repository at tip http://build.golang.org/log/5970f873c651822fe66197c29e359559ad150f03

windows-386-gce at 861af80458e846bc535f262cca23649e703b7885 building image at 5c9906b5359a44e978a11bec5bb6e055bfeb984d

ok      golang.org/x/image/bmp  0.192s
ok      golang.org/x/image/draw 1.058s
?       golang.org/x/image/math/f32 [no test files]
?       golang.org/x/image/math/f64 [no test files]
ok      golang.org/x/image/riff 0.064s
# golang.org/x/image/vp8
run compiler with -v for register allocation sites
..\..\gopath\src\golang.org\x\image\vp8\partition.go:77: internal compiler error: out of fixed registers
ok      golang.org/x/image/tiff 0.231s
?       golang.org/x/image/tiff/lzw [no test files]

Alex
@alexbrainman
Copy link
Member Author

Meant to say "image" (not "exp") repo.

Alex

@bradfitz bradfitz added this to the Go1.5 milestone Jul 29, 2015
@bradfitz bradfitz changed the title cmd/compile: internal compiler error: out of fixed registers x/image/vp8: internal compiler error: out of fixed registers Jul 29, 2015
@bradfitz
Copy link
Contributor

For #11811 (green subrepos), I recommend we just +build this package away for below Go 1.5 if it doesn't compile successfully anyway. No point testing something that's otherwise unusable.

@bradfitz bradfitz changed the title x/image/vp8: internal compiler error: out of fixed registers cmd/compile: internal compiler error: out of fixed registers on 386 Jul 29, 2015
@bradfitz
Copy link
Contributor

Oh, this is all 386 even at tip. I misremembered this only being a Go 1.4 issue.

I suppose we could just say the vp8 package requires 64-bit for now, which is lame, but would get the dashboard "ok" for now and stop hiding other potential test failures.

/cc @rsc for thoughts.

@nigeltao
Copy link
Contributor

FWIW, the vp8 code isn't particularly tricky Go code.

@nigeltao nigeltao assigned rsc and unassigned nigeltao Jul 29, 2015
@rsc
Copy link
Contributor

rsc commented Jul 29, 2015

Not fixing the 386 compiler today. That's Keith's (long-term) job.
https://go-review.googlesource.com/12856

@golang golang locked and limited conversation to collaborators Aug 5, 2016
mrhyperbit23z0d added a commit to mrhyperbit23z0d/bhegde8 that referenced this issue Jun 6, 2022
@mrhyperbit23z0d
vp8: split expression to placate 386 compiler
24b415d 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
GalaxyForcew added a commit to GalaxyForcew/A1bisshy that referenced this issue Jun 6, 2022
@GalaxyForcew
vp8: split expression to placate 386 compiler
be62366 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
yi-ge3 added a commit to yi-ge3/wislie that referenced this issue Jun 6, 2022
@yi-ge3
vp8: split expression to placate 386 compiler
098b0b2 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
balloontmz6 added a commit to balloontmz6/Likewise42l that referenced this issue Jun 6, 2022
@balloontmz6
vp8: split expression to placate 386 compiler
cc8c699 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
@rsc rsc removed their assignment Jun 23, 2022
snapbakkhfbav added a commit to snapbakkhfbav/SayedBaladohr that referenced this issue Oct 6, 2022
@snapbakkhfbav
vp8: split expression to placate 386 compiler
555c809 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
MiderWong5ddop added a commit to MiderWong5ddop/sidie88f that referenced this issue Oct 7, 2022
@MiderWong5ddop
vp8: split expression to placate 386 compiler
ac434e4 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
rorypeckwnt4v added a commit to rorypeckwnt4v/LearnByBhanuPrataph that referenced this issue Oct 7, 2022
@rorypeckwnt4v
vp8: split expression to placate 386 compiler
ae1219a 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
egorovcharenko9 added a commit to egorovcharenko9/RiceBIOC470z that referenced this issue Oct 7, 2022
@egorovcharenko9
vp8: split expression to placate 386 compiler
d3c5adc 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
RafayGhafoorf added a commit to RafayGhafoorf/dustinsand8 that referenced this issue Oct 7, 2022
@RafayGhafoorf
vp8: split expression to placate 386 compiler
d970eba 
Fixes golang/go#11917.

Change-Id: I2eff23bd98be7531d9a115714ca50e45d3064584
Reviewed-on: https://go-review.googlesource.com/12856
Reviewed-by: Ian Lance Taylor <iant@golang.org>
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
FrozenDueToAge
Projects
None yet
Milestone
Go1.5
Development

No branches or pull requests
5 participants
@bradfitz @rsc @nigeltao @gopherbot @alexbrainman