On Tue, Jan 8, 2013 at 12:36 PM, Kenneth Graunke <kenn...@whitecape.org> wrote:
> Clang doesn't support the TLS mode we use, so building Mesa with Clang
> and --enable-glx-tls simply causes every Piglit test under the sun to
> crash when calling glGetString().  Good times.
>
> To avoid catastrophic failure and confusion, simply bail in configure.
> ---
>  configure.ac | 4 ++++
>  1 file changed, 4 insertions(+)
>
> diff --git a/configure.ac b/configure.ac
> index 1414ce8..e7c6d8d 100644
> --- a/configure.ac
> +++ b/configure.ac
> @@ -981,6 +981,10 @@ AC_ARG_ENABLE([glx-tls],
>      [GLX_USE_TLS=no])
>  AC_SUBST(GLX_TLS, ${GLX_USE_TLS})
>
> +if test "x$GLX_USE_TLS" = xyes -a "x$acv_mesa_CLANG" = xyes; then
> +    AC_MSG_ERROR([Clang does not support building GLX with TLS])
> +fi
> +
>  AS_IF([test "x$GLX_USE_TLS" = xyes -a "x$ax_pthread_ok" = xyes],
>        [DEFINES="${DEFINES} -DGLX_USE_TLS -DHAVE_PTHREAD"])
>
> --
> 1.8.1

Reviewed-by: Matt Turner <matts...@gmail.com>
_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to