aboutsummaryrefslogtreecommitdiff
path: root/src/cmd/compile/internal/ssa/export_test.go
diff options
context:
space:
mode:
authorHeschi Kreinick <heschi@google.com>2018-03-13 16:14:52 -0400
committerDavid Chase <drchase@google.com>2018-03-14 20:52:29 +0000
commit44e65f2c94bbae463314382fd77ce690c81b413e (patch)
tree7fc4f552fe4a86637dec51aefb50e17fe73a2b54 /src/cmd/compile/internal/ssa/export_test.go
parentaff222cd185d10400b9177fe26ec06eb647b092d (diff)
downloadgo-44e65f2c94bbae463314382fd77ce690c81b413e.tar.gz
go-44e65f2c94bbae463314382fd77ce690c81b413e.zip
cmd/compile/internal/ssa: track stack-only vars
User variables that cannot be SSA'd, either because their addresses are taken or because they are too large for the decomposition heuristic, do not explicitly appear as operands of SSA values. Instead they are written to directly via the stack pointer. This hid them from the location list generation, which is only interested in the named value table. Fortunately, the lifetime of stack-only variables is delineated by VarDef/VarKill ops, and it's easy enough to turn those into location list bounds. One wrinkle: stack frame information is not explicitly available in the SSA phases, because it's owned by the frontend in AllocFrame. It would be easier if the set of live LocalSlots were returned by that, but this is the minimal change to fix missing variables. Or VarDef/VarKills could appear in NamedValues, which would make this change even easier. Change-Id: Ice6654dad6f9babb0286e95c7ec28594561dc91f Reviewed-on: https://go-review.googlesource.com/100458 Reviewed-by: David Chase <drchase@google.com> Run-TryBot: David Chase <drchase@google.com> TryBot-Result: Gobot Gobot <gobot@golang.org>
Diffstat (limited to 'src/cmd/compile/internal/ssa/export_test.go')
0 files changed, 0 insertions, 0 deletions