-
Notifications
You must be signed in to change notification settings - Fork 17.8k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
misc/cgo/test: link failure with atomics on arm64 #58771
Comments
Might be related to an old bug? #39466 |
Just a note that this is specifically running misc/cgo/test with Looks like for #39466 we disabled this test on arm64. Then in https://go.dev/cl/404296 Cherry fixed the handling of hidden symbols such that internal linking worked again on arm64. But now it is broken again and someone will have to figure out why. Note that this does not affect anybody using Go, unless they are explicitly using |
It looks to me that these object files are supposed to provide the missing symbols ( |
We've not intentionally done anything interesting here, only boring normal flags (
Interestingly:
(We've had an odd bug reported in Gentoo before at https://bugs.gentoo.org/868018 but only for GCC 11, while this is GCC 12, and our GCC 12 doesn't exhibit the issue discussed in that bug, so it might not be related. Just thought I'd mention it on the off chance.) |
Could you share the |
|
What version of Go are you using (
go version
)?Does this issue reproduce with the latest release?
Yes, with 1.20.1
What operating system and processor architecture are you using (
go env
)?go env
OutputWhat did you do?
On an arm64 machine, I've used previous bootstrap of 1.19.5 (located at
/usr/lib/go
, had passed the tests) and compiled and ran the test suite:cd src
GOROOT_BOOTSTRAP=/usr/lib/go GOOS=linux GOARCH=arm64 ./make.bash
PATH="$(dirname ${PWD})/bin:${PATH}" ./run.bash -no-rebuild
What did you expect to see?
Test suite passing as with previous version 1.19.5
What did you see instead?
Test suite failure
The environment where this happened is inside a container, so I can perform extra check or info collection if requested.
The text was updated successfully, but these errors were encountered: