Jeremy Huddleston writes:
>
> On Mar 22, 2011, at 9:13 PM, tom fogal wrote:
> >
> > 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.
>
> I grepped through xserver because it was what broke in tinderbox. I
> did not ch
Dan Nicholson writes:
> On Tue, Mar 22, 2011 at 9:13 PM, tom fogal wrote:
> > The AX_TLS macro should AC_DEFINE_UNQUOTED "TLS". I recommend
> > using that instead of inventing our own define.
>
> Yeah, that seems reasonable. Tom, is that what's done in mesa?
It *should* be what's done in Mesa,
On Mar 22, 2011, at 9:13 PM, tom fogal wrote:
> Jeremy Huddleston 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}
On Tue, Mar 22, 2011 at 9:13 PM, tom fogal wrote:
> Jeremy Huddleston 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_t
Jeremy Huddleston 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
AX_TLS detects when toolchains support __thread or __declspec(thread),
but existing code assumed __thread.
Found-by: Tinderbox
http://tinderbox.x.org/builds/2011-03-22-0007
Regression-from: 82b1eaa6cad20f39dbf15573bdb3d62acbcd91f9
Signed-off-by: Jeremy Huddleston
---
configure.ac|