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/link: Xcode 8.3 dsymutil rejects our binaries #19772

Closed
rsc opened this issue Mar 29, 2017 · 14 comments
Closed

cmd/link: Xcode 8.3 dsymutil rejects our binaries #19772

rsc opened this issue Mar 29, 2017 · 14 comments
Labels
FrozenDueToAge
Milestone
Go1.8.1

Comments

@rsc
Copy link
Contributor

rsc commented Mar 29, 2017

In src/runtime/testdata/testprogcgo, running the final parts of a link by hand:

$ ls -l testprogcgo
-rwxr-xr-x  1 rsc  eng  2008504 Mar 29 14:35 testprogcgo
$ dsymutil -f testprogcgo -o testprogcgo.dw
warning: (x86_64) /tmp/go.o unable to open object file: Invalid data was encountered while parsing the file
$ echo $?
0
$

The resulting binary is malformed.

This is the root cause of #19734, but we'll treat that issue as fixed by the workaround of not trusting dsymutil's exit status when it prints "warnings" (for backport to Go 1.8).

This issue is to fix our object files to be compatible with dsymutil, for Go 1.9.
@rsc rsc added this to the Go1.9 milestone Mar 29, 2017
@rsc rsc changed the title cmd/link: Xcode 8.2 dsymutil rejects our binaries cmd/link: Xcode 8.3 dsymutil rejects our binaries Mar 29, 2017
@rsc rsc mentioned this issue Mar 29, 2017
Closed
@gopherbot
Copy link
Contributor

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

@gregory-m
Copy link
Contributor

I bisected https://github.com/llvm-mirror/llvm
(I definitely don't miss c++ compile times)

The problem with go compiler starts at llvm-mirror/llvm@9a41e59

@gregory-m
Copy link
Contributor

Also please notice apple's and llvm's dsymutil is different binaries.

Apple dsymutil default output is:

error: No input files.

llvml dsymutil default output is:

dsymutil: Not enough positional command line arguments specified!
Must specify at least 1 positional argument: See: ./bin/dsymutil - help

I don't know how Apple use llvm in Xcode. But I can compile go binaries with cgo on mac os 10.12.3 using llvm's dsymutil before 9a41e59, and can't compile with llvm's dsymutil after this change.

@rsc
Copy link
Contributor Author

rsc commented Mar 30, 2017

@gregory-m, thanks for tracking that down. I've got a fix pending and it is definitely related to the alignment of the DWARF segment having changed.

@rsc rsc modified the milestones: Go1.8.1, Go1.9 Mar 30, 2017
@gopherbot
Copy link
Contributor

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

@mattn mattn mentioned this issue Mar 30, 2017
Closed
@tamird
Copy link
Contributor

tamird commented Mar 30, 2017

@rsc could this be backported for 1.8.1?

@dmitshur
Copy link
Contributor

@tamird The current milestone for this issue is Go1.8.1, so I believe that's the plan already.

@tamird
Copy link
Contributor

tamird commented Mar 30, 2017

Heh, yet the issue is closed.

@tamird
Copy link
Contributor

tamird commented Mar 30, 2017

Ah, there's #19734.

@rsc
Copy link
Contributor Author

rsc commented Mar 30, 2017

Be the bot you want to see in the world.

@rsc rsc reopened this Mar 30, 2017
@mcandre
Copy link

mcandre commented Apr 4, 2017

ETA on go 1.8.1 release, so Mac developers can stop worrying about Xcode versions?

@rsc
Copy link
Contributor Author

rsc commented Apr 5, 2017

We hope this week.

@gopherbot
Copy link
Contributor

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

@rsc
Copy link
Contributor Author

rsc commented Apr 5, 2017

Cherry-picked.

@rsc rsc closed this as completed Apr 5, 2017
gopherbot pushed a commit that referenced this issue Apr 5, 2017
@rsc @aclements
[release-branch.go1.8] cmd/link: emit a mach-o dwarf segment that dsy…
2d00430 
…mutil will accept

Right now, at least with Xcode 8.3, we invoke dsymutil and dutifully
copy what it produces back into the binary, but it has actually dropped
all the DWARF information that we wanted, because it didn't like
the look of go.o.

Make it like the look of go.o.

DWARF is tested in other ways, but typically indirectly and not for cgo programs.
Add a direct test, and one that exercises cgo.
This detects missing dwarf information in cgo-using binaries on macOS,
at least with Xcode 8.3, and possibly earlier versions as well.

Fixes #19772.

The backport to Go 1.8 disables TestDWARF on Windows because Windows
DWARF support is new in Go 1.9.

Change-Id: I0082e52c0bc8fc4e289770ec3dc02f39fd61e743
Reviewed-on: https://go-review.googlesource.com/39605
Run-TryBot: Austin Clements <austin@google.com>
TryBot-Result: Gobot Gobot <gobot@golang.org>
Reviewed-by: Russ Cox <rsc@golang.org>
@sigma sigma mentioned this issue May 6, 2017
Merged
7 tasks
@golang golang locked and limited conversation to collaborators Apr 5, 2018
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.8.1
Development

No branches or pull requests
6 participants
@mcandre @rsc @gregory-m @tamird @dmitshur @gopherbot