Mesa (main): glx: __glX_tls_Context should use __THREAD_INITIAL_EXEC

GitLab Mirror gitlab-mirror at kemper.freedesktop.org
Fri Jul 29 13:45:14 UTC 2022


Module: Mesa
Branch: main
Commit: 7d35d560aca3af567974121c1612b1a3eaeb7419
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=7d35d560aca3af567974121c1612b1a3eaeb7419

Author: Yonggang Luo <luoyonggang at gmail.com>
Date:   Thu Jul 28 16:23:37 2022 +0800

glx: __glX_tls_Context should use __THREAD_INITIAL_EXEC

Error message on OSX:
../src/glx/tests/fake_glx_screen.cpp:101:20: error: thread-local declaration of '__glX_tls_Context' with dynamic initialization follows declaration with static initialization
thread_local void *__glX_tls_Context = &dummyContext;
                   ^
../src/glx/glxclient.h:655:36: note: previous declaration is here
extern __THREAD_INITIAL_EXEC void *__glX_tls_Context;

Fixes: be00a7c8acf ("glx: using C++11 keyword thread_local")

Signed-off-by: Yonggang Luo <luoyonggang at gmail.com>
Reviewed-by: Eric Engestrom <eric at igalia.com>
Reviewed-by: Adam Jackson <ajax at redhat.com>
Part-of: <https://gitlab.freedesktop.org/mesa/mesa/-/merge_requests/17784>

---

 src/glx/tests/fake_glx_screen.cpp | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/glx/tests/fake_glx_screen.cpp b/src/glx/tests/fake_glx_screen.cpp
index 46e94d79fa1..9ccbd64f23b 100644
--- a/src/glx/tests/fake_glx_screen.cpp
+++ b/src/glx/tests/fake_glx_screen.cpp
@@ -98,7 +98,7 @@ struct glx_context dummyContext = {
    sizeof(dummyBuffer),
    &dummyVtable
 };
-thread_local void *__glX_tls_Context = &dummyContext;
+__THREAD_INITIAL_EXEC void *__glX_tls_Context = &dummyContext;
 
 #if !defined(USE_ELF_TLS)
 extern "C" struct glx_context *



More information about the mesa-commit mailing list