Commit 3d6d2a63 by Ian Lance Taylor

re PR go/67101 (mprof.goc:408:5: error: calling ‘__builtin_frame_address’ with a…

re PR go/67101 (mprof.goc:408:5: error: calling ‘__builtin_frame_address’ with a nonzero argument is unsafe [-Werror=frame-address])

	PR go/67101

    runtime: Remove call to __builtin_frame_address.
    
    __builtin_frame_address was only supposed to use nonzero arguments
    for debugging purposes.  Calling it with nonzero arguments can have
    unpredictable results and uses are now marked unsafe when
    -Wframe-address is enabled.
    
    Reviewed-on: https://go-review.googlesource.com/13063

From-SVN: r226525
parent 8a921278
2bf7c643a1d2f8503070c8e6cb87852026e32400 d5aad2f400a0f21724e33e4ae48e1583ed8b1a87
The first line of this file holds the git revision number of the last The first line of this file holds the git revision number of the last
merge done from the gofrontend repository. merge done from the gofrontend repository.
...@@ -402,10 +402,9 @@ func ThreadCreateProfile(p Slice) (n int, ok bool) { ...@@ -402,10 +402,9 @@ func ThreadCreateProfile(p Slice) (n int, ok bool) {
} }
func Stack(b Slice, all bool) (n int) { func Stack(b Slice, all bool) (n int) {
byte *pc, *sp; byte *pc;
bool enablegc; bool enablegc;
sp = runtime_getcallersp(&b);
pc = (byte*)(uintptr)runtime_getcallerpc(&b); pc = (byte*)(uintptr)runtime_getcallerpc(&b);
if(all) { if(all) {
...@@ -423,7 +422,6 @@ func Stack(b Slice, all bool) (n int) { ...@@ -423,7 +422,6 @@ func Stack(b Slice, all bool) (n int) {
g->writebuf = (byte*)b.__values; g->writebuf = (byte*)b.__values;
g->writenbuf = b.__count; g->writenbuf = b.__count;
USED(pc); USED(pc);
USED(sp);
runtime_goroutineheader(g); runtime_goroutineheader(g);
runtime_traceback(); runtime_traceback();
runtime_printcreatedby(g); runtime_printcreatedby(g);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment