]> Cypherpunks repositories - gostls13.git/commit
cmd/compile: eliminate NoFramePointer
authorAustin Clements <austin@google.com>
Thu, 25 Jan 2018 16:41:41 +0000 (11:41 -0500)
committerAustin Clements <austin@google.com>
Mon, 12 Feb 2018 21:41:21 +0000 (21:41 +0000)
commita7f73c436dd6fee6a267bdc72b286524a70b685b
tree04fc5aca207ce26511898f7b77809870853306c9
parent9b331189c15c4a96651c3d6842d5bd8ee5b5f462
cmd/compile: eliminate NoFramePointer

The NoFramePointer function flag is no longer used, so this CL
eliminates it. This cleans up some confusion between the compiler's
NoFramePointer flag and obj's NOFRAME flag. NoFramePointer was
intended to eliminate the saved base pointer on x86, but it was
translated into obj's NOFRAME flag. On x86, NOFRAME does mean to omit
the saved base pointer, but on ppc64 and s390x it has a more general
meaning of omitting *everything* from the frame, including the saved
LR and ppc64's "fixed frame". Hence, on ppc64 and s390x there are far
fewer situations where it is safe to set this flag.

Change-Id: If68991310b4d00638128c296bdd57f4ed731b46d
Reviewed-on: https://go-review.googlesource.com/92036
Run-TryBot: Austin Clements <austin@google.com>
Reviewed-by: Cherry Zhang <cherryyz@google.com>
src/cmd/compile/internal/gc/gsubr.go
src/cmd/compile/internal/gc/syntax.go