Skip to content
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.

Sign up for GitHub

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

Jump to bottom

runtime: TestDebugCallUnsafePoint failures with <nil> instead of error on noopt builder since 2022-09-05 #55075

Closed
bcmills opened this issue Sep 14, 2022 · 2 comments
Closed

runtime: TestDebugCallUnsafePoint failures with <nil> instead of error on noopt builder since 2022-09-05 #55075

bcmills opened this issue Sep 14, 2022 · 2 comments
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. FrozenDueToAge WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Milestone
Go1.20

Comments

@bcmills
Copy link
Contributor

bcmills commented Sep 14, 2022

greplogs -l -e 'FAIL: TestDebugCallUnsafePoint.*(?:\n\s+.*)got .*\<nil\>' --since=2022-01-01
2022-09-06T12:35:13-8a32354/linux-amd64-noopt
2022-09-06T11:14:09-1c50484/linux-amd64-noopt
2022-09-05T21:39:28-4c1ca42/linux-amd64-noopt
2022-09-05T21:39:15-e2e69ec/linux-amd64-noopt
2022-09-05T08:28:34-bd5595d/linux-amd64-noopt
2022-09-05T08:12:37-67e6542/linux-amd64-noopt
2022-09-05T08:10:03-40ced0c/linux-amd64-noopt
2022-09-05T08:08:24-af7f417/linux-amd64-noopt
2022-09-05T08:08:18-4ad55cd/linux-amd64-noopt
2022-09-05T08:08:09-02700e5/linux-amd64-noopt
2022-09-05T08:07:56-7d83d32/linux-amd64-noopt
2022-09-05T08:07:47-357b922/linux-amd64-noopt

See previously #36110.
@gopherbot gopherbot added the compiler/runtime Issues related to the Go compiler and/or runtime. label Sep 14, 2022
@bcmills
Copy link
Contributor Author

bcmills commented Sep 14, 2022

@golang/runtime, it looks like this may have been fixed around Sept. 7, but I can't find an obvious fix CL. Do we know for certain that this is fixed?

@bcmills bcmills added the WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided. label Sep 14, 2022
@bcmills bcmills added this to the Go1.20 milestone Sep 14, 2022
@cuonglm
Copy link
Member

cuonglm commented Sep 14, 2022

I think this causes by https://go-review.googlesource.com/c/go/+/427135, the CL then be reverted.

@bcmills bcmills mentioned this issue Sep 14, 2022
Closed
@bcmills bcmills closed this as completed Sep 14, 2022
@golang golang locked and limited conversation to collaborators Sep 14, 2023
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
compiler/runtime Issues related to the Go compiler and/or runtime. FrozenDueToAge WaitingForInfo Issue is not actionable because of missing required information, which needs to be provided.
Projects
None yet
Milestone
Go1.20
Development

No branches or pull requests
3 participants
@cuonglm @bcmills @gopherbot