-
-
Notifications
You must be signed in to change notification settings - Fork 0
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
research/pointer-params/ #4
Comments
@Yanwenjiepy 感谢指出,应该是个笔误,已修正。 |
Hello, I found your article and began to conduct my research, at the moment I think that there is a connection not with the size of the structure, but with the number of fields that it has: Modified generator source code: Gist |
@xorcare Thanks for a follow-up to this article. The article was written at Go 1.16 and back then there was no register-based calling convention, and inline optimization was also not as good as 1.17. If you run the experiment on 1.17, the conclusion might be different. Still, I think the conclusion of the article would need some sort of update. Thanks again for bringing this up to discussion 👍 |
JFYI. Environment information:
|
@xorcare Then it makes some sense. If switch to go 1.16, you may reproduce the article's conclusion. |
666 |
Pointers Might Not be Ideal as Arguments
https://golang.design/research/pointer-params/
The text was updated successfully, but these errors were encountered: