Comment 1:

Here is a smaller example:
mwhudson@narsil:mwhudson$ cd $GOPATH/src/mwhudson
mwhudson@narsil:mwhudson$ cat hashfun/hashfun.go
package hashfun
// this type must not be exported
// it must also be complex enough to get a $hash function generated
type unexported struct {
    a int
    b bool
}
// now embed this type in a struct that is exported
type Struct struct {
    unexported
}
mwhudson@narsil:mwhudson$ cat wtf/wtf_test.go
package wtf
import (
    "mwhudson/hashfun"
    "testing"
)
type l hashfun.Struct
func TestAll(t *testing.T) {
}
mwhudson@narsil:mwhudson$ go test -compiler gccgo mwhudson/wtf
# testmain
/tmp/go-build219194897/mwhudson/wtf/_test/mwhudson/libwtf.a(wtf.o): In function
`wtf.l$hash':
/home/mwhudson/goplay/src/mwhudson/wtf/wtf_test.go:11: undefined reference to
`mwhudson_hashfun.unexported$hash'
collect2: error: ld returned 1 exit status
FAIL    mwhudson/wtf [build failed]
It seems that the hash function for an unexported type is inconsistently named
(s/unexported/Exported/ and it works).  Bit confused as to why compiling wtf_test.go
needs to know this name, but don't know how this is supposed to work.

Comment 2:

oh, it's because when you say "type l hashfun.Struct", gccgo generates a new version of
l$hash (that is presumably the same as hashfun.Struct$hash).  Could/should it not just
emit a reference to hashfun.Struct$hash instead?

Comment 3:

Labels changed: added priority-soon, gccgo, removed priority-triage.

Status changed to Accepted.

Comment 4:

Fixed on tip.
Thanks for the test case.

Status changed to Fixed.