-
Notifications
You must be signed in to change notification settings - Fork 18k
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
cmd/go: pass -gcflags after other flags generated by the go command #47682
Comments
If I understand right, the exact issue this report is about is as follows. What did you do?
What did you expect to see?For
What did you see instead?Setting -lang via
Including
The later of the two In this message, @ianlancetaylor suggested the order may be a bug. If I understand right, this issue is about cmd/go, not cmd/compile, since it's the go command that decides the order of arguments for cmd/compile. CC @bcmills, @ianlancetaylor, @mknyszek. |
Can we just simply swap the order of go/src/cmd/go/internal/work/gc.go Line 160 in 7eaabae
The name |
I don't think overriding the language version with
Do we know if the order causes problems anywhere else? Changing this kind of thing may break someone, so I'm not sure we should change it unless passing
The |
My opinion is that people in general shouldn't use |
@ianlancetaylor That sounds reasonable. I've changed the issue title to match. |
Change https://golang.org/cl/344574 mentions this issue: |
Change https://golang.org/cl/344909 mentions this issue: |
What version of Go are you using (
go version
)?What did you do?
https://groups.google.com/g/golang-nuts/c/lhedc8YaWlM/m/R4Q3xuhHBAAJ
What did you expect to see?
Use go1.17 as feature set instead.
What did you see instead?
gc chooses go1.16 as language version finally.
The text was updated successfully, but these errors were encountered: