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: binary exporter debug flag is broken #20684

Closed
ALTree opened this issue Jun 15, 2017 · 4 comments
Closed

cmd/compile: binary exporter debug flag is broken #20684

ALTree opened this issue Jun 15, 2017 · 4 comments
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Milestone

Comments

@ALTree
Copy link
Member

ALTree commented Jun 15, 2017

$ gotip version
go version devel +e7c650bca5 Thu Jun 15 05:57:29 2017 +0000 linux/amd64

There's a debug flag in bexport.go

// NOTE: This flag is the first flag to enable if importing dies because of
// (suspected) format errors, and whenever a change is made to the format.
const debugFormat = false // default: false

but it seems to be broken. Setting it to true and then running make.bash gives:

[...]

##### Building go_bootstrap for host, linux/amd64.
runtime/internal/sys
<autogenerated>:1:0: internal compiler error: incorrect marker: got t; want p (pos = 32)

goroutine 1 [running]:
runtime/debug.Stack(0x0, 0x0, 0x0)
	/home/adonizetti/go/src/runtime/debug/stack.go:24 +0x79
bootstrap/cmd/compile/internal/gc.Fatalf(0x9625be, 0x2c, 0xc4203890c8, 0x3, 0x3)
	/home/adonizetti/go/src/cmd/compile/internal/gc/subr.go:181 +0x24c
bootstrap/cmd/compile/internal/gc.(*importer).formatErrorf(0xc42032fa00, 0x9625be, 0x2c, 0xc4203890c8, 0x3, 0x3)
	/home/adonizetti/go/src/cmd/compile/internal/gc/bimport.go:237 +0x66
bootstrap/cmd/compile/internal/gc.(*importer).marker(0xc42032fa00, 0xc420389170)
	/home/adonizetti/go/src/cmd/compile/internal/gc/bimport.go:1313 +0x26d
bootstrap/cmd/compile/internal/gc.(*importer).path(0xc42032fa00, 0xc42040e760, 0x3)
	/home/adonizetti/go/src/cmd/compile/internal/gc/bimport.go:424 +0x2ca
bootstrap/cmd/compile/internal/gc.(*importer).pkg(0xc42032fa00, 0xc420364f00)
	/home/adonizetti/go/src/cmd/compile/internal/gc/bimport.go:277 +0xd3
bootstrap/cmd/compile/internal/gc.Import(0xc420316aa0, 0xc42038d860)
	/home/adonizetti/go/src/cmd/compile/internal/gc/bimport.go:118 +0x396
bootstrap/cmd/compile/internal/gc.dumpexport.func1()
	/home/adonizetti/go/src/cmd/compile/internal/gc/export.go:177 +0x8e
bootstrap/cmd/compile/internal/types.CleanroomDo(0xc420389788)
	/home/adonizetti/go/src/cmd/compile/internal/types/pkg.go:133 +0x78
bootstrap/cmd/compile/internal/gc.dumpexport()
	/home/adonizetti/go/src/cmd/compile/internal/gc/export.go:178 +0x2b4
bootstrap/cmd/compile/internal/gc.dumpobj1(0x7ffca663a1f5, 0x37, 0x3)
	/home/adonizetti/go/src/cmd/compile/internal/gc/obj.go:95 +0x81b
bootstrap/cmd/compile/internal/gc.dumpobj()
	/home/adonizetti/go/src/cmd/compile/internal/gc/obj.go:51 +0x61
bootstrap/cmd/compile/internal/gc.Main(0x968fc0)
	/home/adonizetti/go/src/cmd/compile/internal/gc/main.go:631 +0x2003
main.main()
	/home/adonizetti/go/src/cmd/compile/main.go:49 +0x9a
@ALTree
Copy link
Member Author

ALTree commented Jun 15, 2017

cc @griesemer

@ALTree ALTree added the NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one. label Jun 15, 2017
@bradfitz bradfitz added this to the Go1.9Maybe milestone Jun 15, 2017
@bradfitz
Copy link
Contributor

bradfitz commented Jun 15, 2017

I'll conservatively flag this Go 1.9 in case it implies a larger problem.

@griesemer
Copy link
Contributor

Minor discrepancy in redundant code used for debugging. Does not affect anything else but will fix.

@gopherbot
Copy link

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

@griesemer griesemer self-assigned this Jun 15, 2017
@griesemer griesemer modified the milestones: Go1.9, Go1.9Maybe Jun 15, 2017
@golang golang locked and limited conversation to collaborators Jun 15, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
FrozenDueToAge NeedsInvestigation Someone must examine and confirm this is a valid issue and not a duplicate of an existing one.
Projects
None yet
Development

No branches or pull requests

4 participants