• Russ Cox's avatar
    runtime: fix nacl build · f9b9ecc5
    Russ Cox authored
    The NaCl "system calls" were assumed to have a compatible
    return convention with the C compiler, and we were using
    tail jumps to those functions. Don't do that anymore.
    
    Correct mistake introduced in newstackcall duringconversion
    from (SP) to (FP) notation. (Actually this fix, in asm_amd64p32.s,
    slipped into the C compiler change, but update the name to
    match what go vet wants.)
    
    Correct computation of caller stack pointer in morestack:
    on amd64p32, the saved PC is the size of a uintreg, not uintptr.
    This may not matter, since it's been like this for a while,
    but uintreg is the correct one. (And on non-NaCl they are the same.)
    
    This will allow the NaCl build to get much farther.
    It will probably still not work completely.
    There's a bug in 6l that needs fixing too.
    
    TBR=minux
    CC=golang-codereviews
    https://golang.org/cl/134990043
    f9b9ecc5
stack.c 32.4 KB