diff options
| author | Russ Cox <rsc@golang.org> | 2014-09-12 07:46:11 -0400 |
|---|---|---|
| committer | Russ Cox <rsc@golang.org> | 2014-09-12 07:46:11 -0400 |
| commit | e844f53a0198e81b359d198fc0dcf15cf01d6ed1 (patch) | |
| tree | 61fa2ae1685cb6f7ebc6d89d95d35dd0b830bd31 /src/runtime/runtime.h | |
| parent | a7c6d89166fd7bf6c9af6013cbfaa21971ba28f0 (diff) | |
| download | go-e844f53a0198e81b359d198fc0dcf15cf01d6ed1.tar.xz | |
runtime: stop scanning stack frames/args conservatively
The goal here is to commit fully to having precise information
about stack frames. If we need information we don't have,
crash instead of assuming we should scan conservatively.
Since the stack copying assumes fully precise information,
any crashes during garbage collection that are introduced by
this CL are crashes that could have happened during stack
copying instead. Those are harder to find because stacks are
copied much less often than the garbage collector is invoked.
In service of that goal, remove ARGSIZE macros from
asm_*.s, change switchtoM to have no arguments
(it doesn't have any live arguments), and add
args and locals information to some frames that
can call back into Go.
LGTM=khr
R=khr, rlh
CC=golang-codereviews
https://golang.org/cl/137540043
Diffstat (limited to 'src/runtime/runtime.h')
| -rw-r--r-- | src/runtime/runtime.h | 1 |
1 files changed, 0 insertions, 1 deletions
diff --git a/src/runtime/runtime.h b/src/runtime/runtime.h index 6300b83c97..8c2b09b317 100644 --- a/src/runtime/runtime.h +++ b/src/runtime/runtime.h @@ -806,7 +806,6 @@ void runtime·signalstack(byte*, int32); void runtime·symtabinit(void); Func* runtime·findfunc(uintptr); int32 runtime·funcline(Func*, uintptr, String*); -int32 runtime·funcarglen(Func*, uintptr); int32 runtime·funcspdelta(Func*, uintptr); int8* runtime·funcname(Func*); int32 runtime·pcdatavalue(Func*, int32, uintptr); |