Jeremy Huddleston <[email protected]> writes:
> AX_TLS detects when toolchains support __thread or
> __declspec(thread), but existing code assumed __thread.
Good find.
> configure.ac | 1 +
[snip]
> + AC_DEFINE(__XSERVER_TLS, ${ac_cv_tls}, [Thread local storage directive]
The AX_TLS macro should AC_DEFINE_UNQUOTED "TLS". I recommend using
that instead of inventing our own define.
I haven't looked to see if we have __thread anywhere else that should
go away in favor of the macro, but I assume you have.
Other than those little nitpicks:
Reviewed-by: Tom Fogal <[email protected]>
Thanks,
-tom
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
