]> Cypherpunks repositories - gostls13.git/commit
cmd/internal/obj/arm64: reject misaligned stack frames, except empty frames
authorRuss Cox <rsc@golang.org>
Wed, 29 Jul 2015 19:15:03 +0000 (15:15 -0400)
committerRuss Cox <rsc@golang.org>
Wed, 29 Jul 2015 21:37:12 +0000 (21:37 +0000)
commit034a10d44c8f83d2c36ab4e242c14e9744c54949
tree052b319ed96a465814d80397f89ffa74b14cb071
parent767e065809937229767bb3496fdce61b74b815e1
cmd/internal/obj/arm64: reject misaligned stack frames, except empty frames

The layout code has to date insisted on stack frames that are 16-aligned
including the saved LR, and it ensured this by growing the frame itself.
This breaks code that refers to values near the top of the frame by positive
offset from SP, and in general it's too magical: if you see TEXT xxx, $N,
you expect that the frame size is actually N, not sometimes N and sometimes N+8.

This led to a serious bug in the compiler where ambiguously live values
were not being zeroed correctly, which in turn triggered an assertion
in the GC about finding only valid pointers. The compiler has been
fixed to always emit aligned frames, and the hand-written assembly
has also been fixed.

Now that everything is aligned, make unaligned an error instead of
something to "fix" silently.

For #9880.

Change-Id: I05f01a9df174d64b37fa19b36a6b6c5f18d5ba2d
Reviewed-on: https://go-review.googlesource.com/12848
Reviewed-by: Austin Clements <austin@google.com>
src/cmd/internal/obj/arm64/obj7.go
test/nosplit.go