reflect: func created w/MakeFunc returns mangled value when used as a parameter to another func via Call on ARM

[thempatel]

What version of Go are you using (go version)?

$ go version
go version go1.18 darwin/arm64

Does this issue reproduce with the latest release?

Yes.

What operating system and processor architecture are you using (go env)?

OS X 12.4 (Monterey) Apple M1 Pro (ARM)

go env Output

$ go env
GO111MODULE=""
GOARCH="arm64"
GOBIN=""
GOCACHE="/Users/milan/Library/Caches/go-build"
GOENV="/Users/milan/Library/Application Support/go/env"
GOEXE=""
GOEXPERIMENT=""
GOFLAGS=""
GOHOSTARCH="arm64"
GOHOSTOS="darwin"
GOINSECURE=""
GOMODCACHE="/Users/milan/go/pkg/mod"
GONOPROXY=""
GONOSUMDB=""
GOOS="darwin"
GOPATH="/Users/milan/go"
GOPRIVATE=""
GOPROXY="https://proxy.golang.org,direct"
GOROOT="/Users/milan/src/fsdev/tools/go"
GOSUMDB="sum.golang.org"
GOTMPDIR=""
GOTOOLDIR="/Users/milan/src/fsdev/tools/go/pkg/tool/darwin_arm64"
GOVCS=""
GOVERSION="go1.18"
GCCGO="gccgo"
AR="ar"
CC="clang"
CXX="clang++"
CGO_ENABLED="1"
GOMOD="/Users/milan/src/go-reflect-issue/go.mod"
GOWORK=""
CGO_CFLAGS="-g -O2"
CGO_CPPFLAGS=""
CGO_CXXFLAGS="-g -O2"
CGO_FFLAGS="-g -O2"
CGO_LDFLAGS="-g -O2"
PKG_CONFIG="pkg-config"
GOGCCFLAGS="-fPIC -arch arm64 -pthread -fno-caret-diagnostics -Qunused-arguments -fmessage-length=0 -fdebug-prefix-map=/var/folders/wm/x3qn31vj0hgg60d8s9m3brlh0000gq/T/go-build2169734258=/tmp/go-build -gno-record-gcc-switches -fno-common"

What did you do?

https://go.dev/play/p/Vcob_D0J733

What did you expect to see?

Both tests should pass on ARM.

What did you see instead?

Second test fails, i.e the one that tests calling a function with reflect.

Prose

As the title indicates, on ARM, it appears that MakeFunc results in a func that returns a mangled value when used as a parameter to another function that was invoked using Call. This problem does not occur on amd64 architectures, it is currently, at the very least, a problem on ARM architectures.

[seankhliao]

cc @golang/runtime

[cherrymui]

This fails with Go 1.18 but passes on ARM64 on tip. Looks the same as #52788 .

[thempatel]

@cherrymui these look related! Great. Can you comment on when the fixes will be released? Is it a cherry-pick candidate or will we need to wait for go1.19?

[cherrymui]

I opened backport issues for #52788. It may be included in the next Go 1.18 and 1.17 releases in early July.

I think we can close this as a dup. Thanks.

[thempatel]

thank you!