Re: [Mesa-dev] [PATCH v3 2/2] GLX_TLS: use TLS macros when define those TLS variables.

2012-03-19 Thread Zhigang Gong
zhigang.gong=linux.intel.com@lists.freedeskto > p.org] On Behalf Of Brian Paul > Sent: Thursday, February 16, 2012 10:38 PM > To: Kenneth Graunke > Cc: mesa-dev@lists.freedesktop.org > Subject: Re: [Mesa-dev] [PATCH v3 2/2] GLX_TLS: use TLS macros when > define those TLS variables. >

Re: [Mesa-dev] [PATCH v3 2/2] GLX_TLS: use TLS macros when define those TLS variables.

2012-02-16 Thread Brian Paul
On 02/16/2012 01:42 AM, Kenneth Graunke wrote: On 02/15/2012 04:58 AM, tf (mobile) wrote: Personally i don't care much about non-autoconf builds, but as this relies on a -DTLS= setting during configuration time, it will break non-ac builds (which need tls), no? Other than that, LGTM. -tom Th

Re: [Mesa-dev] [PATCH v3 2/2] GLX_TLS: use TLS macros when define those TLS variables.

2012-02-16 Thread Kenneth Graunke
On 02/15/2012 04:58 AM, tf (mobile) wrote: Personally i don't care much about non-autoconf builds, but as this relies on a -DTLS= setting during configuration time, it will break non-ac builds (which need tls), no? Other than that, LGTM. -tom That's okay, we've already broken non-automake b

Re: [Mesa-dev] [PATCH v3 2/2] GLX_TLS: use TLS macros when define those TLS variables.

2012-02-15 Thread tf (mobile)
Personally i don't care much about non-autoconf builds, but as this relies on a -DTLS= setting during configuration time, it will break non-ac builds (which need tls), no? Other than that, LGTM. -tom On 15.02.2012, at 12:41, zhigang.g...@linux.intel.com wrote: > From: Zhigang Gong > > Use t

[Mesa-dev] [PATCH v3 2/2] GLX_TLS: use TLS macros when define those TLS variables.

2012-02-15 Thread zhigang . gong
From: Zhigang Gong Use the properate way detected in autoconf stage to define those TLS variables rather than using hard coded "attribute..." Signed-off-by: Zhigang Gong --- src/egl/main/eglcurrent.c |3 +-- src/glx/glxclient.h|3 +-- src/glx/glxcurrent