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/addr2line: new TestAddr2Line fails on plan9 #7947
Labels
Comments
This issue was updated by revision 3b3e5ea. LGTM=bradfitz R=golang-codereviews, bradfitz CC=golang-codereviews https://golang.org/cl/100180043 |
Owner changed to @0intro. |
CL https://golang.org/cl/91500044 mentions this issue. |
Support for Go Plan 9 binaries has been implemented in cmd/addr2line and cmd/objdump. The tests have been enabled again. However, TestDisasm is failing: === RUN TestDisasm --- FAIL: TestDisasm (3.95 seconds) objdump_test.go:187: disassembly missing 'fmthello.go:6' objdump_test.go:187: disassembly missing 'fmthello.go:6' objdump_test.go:187: disassembly missing 'JMP main.main(SB)' objdump_test.go:187: disassembly missing 'CALL fmt.Println(SB)' objdump_test.go:187: disassembly missing 'RET' objdump_test.go:192: full disassembly: TEXT main.main(SB) /usr/go/src/cmd/objdump/testdata/fmthello.go fmthello.go:5 0x1020 8b0df0b00f00 MOVL 0xfb0f0(IP), CX fmthello.go:5 0x1026 8b89f8ffffff MOVL 0xfffffff8(CX), CX fmthello.go:5 0x102c 3b21 CMPL 0(CX), SP fmthello.go:5 0x102e 770b JA 0x103b |
I've attached a comparison of objdump on fmthello.go between Plan 9 on Linux on 386. It seems a part of main.main is missing on the Plan 9 binary. Attachments:
|
CL https://golang.org/cl/93520045 mentions this issue. |
This issue was closed by revision eb34288. Status changed to Fixed. |
wheatman
pushed a commit
to wheatman/go-akaros
that referenced
this issue
Jun 25, 2018
Ignore symbols that aren't text, data, or bss since they cause problems when dissassembling instructions with small immediate values. Before: build.go:142 0x10ee 83ec50 SUBL $text/template/parse.autotmp_1293(SB), SP After: build.go:142 0x10ee 83ec50 SUBL $0x50, SP Fixes golang#7947. LGTM=rsc R=rsc, 0intro CC=golang-codereviews https://golang.org/cl/93520045
This issue was closed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
The text was updated successfully, but these errors were encountered: