You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
There's no indication why this code was added. I suspect it is not necessary (maybe it was, but is no longer). This issue is to investigate why we do this and whether we can remove it.
Reported on go-nuts by Eric from arm.
For a simple repro, compile
package main
func g()
func f(p, q *int16) {
x := *p
y := *q
g()
*p = y
*q = x
}
The autotmps should be 2 bytes apart, not 8. They are 2 bytes apart on amd64, but not arm64.
The text was updated successfully, but these errors were encountered:
In cmd/compile/internal/gc/pgen.go:185, we do:
This code originated in commit 2ac375b (then called cmd/gc/pgen.c:474, committed by Luuk van Dijk).
There's no indication why this code was added. I suspect it is not necessary (maybe it was, but is no longer). This issue is to investigate why we do this and whether we can remove it.
Reported on go-nuts by Eric from arm.
For a simple repro, compile
The autotmps should be 2 bytes apart, not 8. They are 2 bytes apart on amd64, but not arm64.
The text was updated successfully, but these errors were encountered: