cmd/8g: out of registers in go.net subrepo since rev. 3ae4607a43ff

[mikioh]

See http://build.golang.org/log/30767515b426ba3acc4070593feb4e4219a8cd1d.

[mikioh]

Comment 1:

Labels changed: added repo-main, release-go1.3.

[remyoudompheng]

Comment 2:

The naïve fix does not work.
diff -r 4ab152b54171 src/cmd/8g/cgen.c
--- a/src/cmd/8g/cgen.c Mon Feb 24 22:31:01 2014 -0500
+++ b/src/cmd/8g/cgen.c Tue Feb 25 08:50:20 2014 +0100
@@ -920,6 +920,9 @@
    if(n == N)
        n = nodbool(1);
 
+   while(n->op == OCONVNOP)
+       n = n->left;
+
    if(n->ninit != nil)
        genlist(n->ninit);
 
gives me:
$ GOARCH=386 /opt/remy/go/pkg/tool/linux_amd64/8g -pack ./control.go ./control_linux.go
./dgramopt_posix.go ./doc.go ./endpoint.go ./genericopt_posix.go ./header.go ./helper.go
./helper_posix.go ./helper_unix.go ./iana.go ./icmp.go ./packet.go ./payload.go
./sockopt_linux.go ./sockopt_unix.go
./control.go:1: internal compiler error: temp used and not set: 17183
(./control.go:1)    MOVL    "".autotmp_1264+0(SP),BX
I wonder if there is a hidden second bug.

[remyoudompheng]

Comment 3:

Status changed to Accepted.

[remyoudompheng]

Comment 4:

Issue #7494 has been merged into this issue.

[ality]

Comment 5:

Here's a minimal reproducer:
package p
import "net"
type T1 struct {
    T2
    T3
    T4
}
type T2 struct {
    net.Conn
}
type T3 struct {
    net.PacketConn
}
type T4 struct {
    net.PacketConn
    T5
}
type T5 struct {
    x int
    T6
}
type T6 struct {
    y, z int
}

[remyoudompheng]

Comment 6:

This issue was closed by revision 0285d2b.

Status changed to Fixed.

[mikioh]

Comment 7:

Yay!