Sounds fine. We've done that elsewhere in std already.

For instance:

• https://go-review.googlesource.com/c/go/+/6701/
• https://go-review.googlesource.com/c/go/+/8192/
• https://go-review.googlesource.com/c/go/+/28484/

@josharian I'm curious; is there a reason you're looking into the performance of os/exec recently?

In some circumstances it was showed up in go-fuzz pprof output. See also dvyukov/go-fuzz#215.

I started on this and then started to wonder whether we should do this trick in the compiler for small ints in interfaces instead, like we do with single-byte values. We have talked about that in the past and worried about binary size and weird perf cliffs. But this change would add more to binary size, and almost everyone uses syscall, and doing it in the compiler would benefit other code.

(Note to self; on phone.)

A narrower fix would be to add a ByteErrno, applicable only for byte-sized errnos, that delegates to syscall.Errno for errors strings.

A narrower fix would be to add a ByteErrno

No adding exported stuff to syscall, though. And we can't change the type on people anyway.

Btw, we already have:

https://github.com/golang/go/blob/8f9902d/src/syscall/syscall_unix.go#L139

// errnoErr returns common boxed Errno values, to prevent
// allocations at runtime.
func errnoErr(e Errno) error {
	switch e {
	case 0:
		return nil
	case EAGAIN:
		return errEAGAIN
	case EINVAL:
		return errEINVAL
	case ENOENT:
		return errENOENT
	}
	return e
}

... feel free to add to that. Or change its implementation to the array thing.

No adding exported stuff to syscall, though. And we can't change the type on people anyway.

I was thinking of it just for this particular use, which never makes it to the outside world.

errnoErr is perfect but unexported and in the wrong package. I suppose I could export it and move it to some internal package.

errnoErr is perfect but unexported and in the wrong package. I suppose I could export it and move it to some internal package.

SGTM

Change https://golang.org/cl/164971 mentions this issue: internal/poll: copy and use errnoErr to avoid allocations