diff options
author | David Chase <drchase@google.com> | 2021-02-17 10:38:03 -0500 |
---|---|---|
committer | David Chase <drchase@google.com> | 2021-03-03 15:00:48 +0000 |
commit | 9f33dc3ca1b7b6bdb1a8e83c24d490f579bbbdc8 (patch) | |
tree | 2b3ff2bdfd9e48e78cdd1570c86c2957a093b5b1 /src/cmd/compile/internal/amd64 | |
parent | c4e3f6c4c78f52060d409a549b83b72644069137 (diff) | |
download | go-9f33dc3ca1b7b6bdb1a8e83c24d490f579bbbdc8.tar.gz go-9f33dc3ca1b7b6bdb1a8e83c24d490f579bbbdc8.zip |
cmd/compile: handle aggregate OpArg in registers
Also handles case where OpArg does not escape but has its address
taken.
May have exposed a lurking bug in 1.16 expandCalls,
if e.g., loading len(someArrayOfstructThing[0].secondStringField)
from a local. Maybe.
For #40724.
Change-Id: I0298c4ad5d652b5e3d7ed6a62095d59e2d8819c7
Reviewed-on: https://go-review.googlesource.com/c/go/+/293396
Trust: David Chase <drchase@google.com>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
Diffstat (limited to 'src/cmd/compile/internal/amd64')
-rw-r--r-- | src/cmd/compile/internal/amd64/ssa.go | 2 |
1 files changed, 0 insertions, 2 deletions
diff --git a/src/cmd/compile/internal/amd64/ssa.go b/src/cmd/compile/internal/amd64/ssa.go index 3c43a1d41b..d83d78f080 100644 --- a/src/cmd/compile/internal/amd64/ssa.go +++ b/src/cmd/compile/internal/amd64/ssa.go @@ -980,8 +980,6 @@ func ssaGenValue(s *ssagen.State, v *ssa.Value) { ssagen.AddAux(&p.From, v) p.To.Type = obj.TYPE_REG p.To.Reg = v.Reg() - case ssa.OpArgIntReg, ssa.OpArgFloatReg: - ssagen.CheckArgReg(v) case ssa.OpAMD64LoweredGetClosurePtr: // Closure pointer is DX. ssagen.CheckLoweredGetClosurePtr(v) |