OpenE2K
/
gcc
2
0
Fork 0
Code Issues Pull Requests Projects Releases Wiki Activity

runtime: expose the g variable 

Currently, getg is implemented in C, which loads the thread-local
    g variable. The g variable is declared static in C.
    
    This CL exposes the g variable, so it can be accessed from the Go
    side. This allows the Go compiler to inline getg calls to direct
    access of g.
    
    Currently, the actual inlining is only implemented in the gollvm
    compiler. The g variable is thread-local and the compiler backend
    may choose to cache the TLS address in a register or on stack. If
    a thread switch happens the cache may become invalid. I don't
    know how to disable the TLS address cache in gccgo, therefore
    the inlining of getg is not implemented. In the future gccgo may
    gain this if we know how to do it safely.
    
    Reviewed-on: https://go-review.googlesource.com/c/gofrontend/+/186238

From-SVN: r273499
This commit is contained in:
Ian Lance Taylor 2019-07-15 21:17:16 +00:00
parent 2d94f7dea9
commit 4e62f891cd
2 changed files with 3 additions and 3 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
gcc/go/gofrontend/MERGE
View File

@ -1,4 +1,4 @@
70ceba5e95716653b9f829a457a44a829175d4da
0e51b7e9c03c6f6bc3d06343f2050f17349ccdc3
The first line of this file holds the git revision number of the last
merge done from the gofrontend repository.

4
libgo/runtime/proc.c
View File

@ -65,7 +65,7 @@ static void gscanstack(G*);
#define __thread
#endif
static __thread G *g;
__thread G *g __asm__(GOSYM_PREFIX "runtime.g");
#ifndef SETCONTEXT_CLOBBERS_TLS
@ -320,7 +320,7 @@ runtime_mcall(FuncVal *fv)
if(gp != nil) {
#ifdef USING_SPLIT_STACK
__splitstack_getcontext((void*)(&g->stackcontext[0]));
__splitstack_getcontext((void*)(&gp->stackcontext[0]));
#else
// We have to point to an address on the stack that is
// below the saved registers.