diff options
author | Russ Cox <rsc@golang.org> | 2013-10-30 10:24:42 -0400 |
---|---|---|
committer | Russ Cox <rsc@golang.org> | 2013-10-30 10:24:42 -0400 |
commit | 961767536de65136b04eeb14649613e7c7fc943d (patch) | |
tree | 34ec2c74f6f4fcfe3ce103811a2c5b07ee1d3e23 /src/cmd/cgo | |
parent | 10658dab2f66cede5c15e1c0eae6df2718132f8a (diff) | |
download | go-961767536de65136b04eeb14649613e7c7fc943d.tar.gz |
cmd/cgo: accept extra leading _ on __cgodebug_data for all object formats
The current Windows build breakage appears to be because
the Windows code should be looking for __cgodebug_data
not ___cgodebug_data. Dodge the question everywhere by
accepting both.
R=golang-dev, iant
CC=golang-dev
https://codereview.appspot.com/19780043
Diffstat (limited to 'src/cmd/cgo')
-rw-r--r-- | src/cmd/cgo/gcc.go | 12 |
1 files changed, 8 insertions, 4 deletions
diff --git a/src/cmd/cgo/gcc.go b/src/cmd/cgo/gcc.go index 60d420b21..f2a109d34 100644 --- a/src/cmd/cgo/gcc.go +++ b/src/cmd/cgo/gcc.go @@ -780,6 +780,11 @@ func (p *Package) gccCmd() []string { func (p *Package) gccDebug(stdin []byte) (*dwarf.Data, binary.ByteOrder, []byte) { runGcc(stdin, p.gccCmd()) + isDebugData := func(s string) bool { + // Some systems use leading _ to denote non-assembly symbols. + return s == "__cgodebug_data" || s == "___cgodebug_data" + } + if f, err := macho.Open(gccTmp()); err == nil { defer f.Close() d, err := f.DWARF() @@ -790,8 +795,7 @@ func (p *Package) gccDebug(stdin []byte) (*dwarf.Data, binary.ByteOrder, []byte) if f.Symtab != nil { for i := range f.Symtab.Syms { s := &f.Symtab.Syms[i] - // Mach-O still uses a leading _ to denote non-assembly symbols. - if s.Name == "_"+"__cgodebug_data" { + if isDebugData(s.Name) { // Found it. Now find data section. if i := int(s.Sect) - 1; 0 <= i && i < len(f.Sections) { sect := f.Sections[i] @@ -818,7 +822,7 @@ func (p *Package) gccDebug(stdin []byte) (*dwarf.Data, binary.ByteOrder, []byte) if err == nil { for i := range symtab { s := &symtab[i] - if s.Name == "__cgodebug_data" { + if isDebugData(s.Name) { // Found it. Now find data section. if i := int(s.Section); 0 <= i && i < len(f.Sections) { sect := f.Sections[i] @@ -842,7 +846,7 @@ func (p *Package) gccDebug(stdin []byte) (*dwarf.Data, binary.ByteOrder, []byte) } var data []byte for _, s := range f.Symbols { - if s.Name == "_"+"__cgodebug_data" { + if isDebugData(s.Name) { if i := int(s.SectionNumber) - 1; 0 <= i && i < len(f.Sections) { sect := f.Sections[i] if s.Value < sect.Size { |