Reviewed-by: Tim Rowley <timothy.o.row...@intel.com>

> On Jul 21, 2017, at 1:05 PM, Emil Velikov <emil.l.veli...@gmail.com> wrote:
> 
> From: Emil Velikov <emil.veli...@collabora.com>
> 
> Seems like the backends have been using pthreads since day one, yet
> we've been missing the link.
> 
> With later commit we'll fix a typo, hence the libraries will be build
> with -Wl,no-undefined, aka failing the build on unresolved symbols.
> 
> v2: Split from a larger patch.
> 
> Cc: mesa-sta...@lists.freedesktop.org
> Cc: Bruce Cherniak <bruce.chern...@intel.com>
> Cc: Tim Rowley <timothy.o.row...@intel.com>
> Cc: Laurent Carlier <lordhea...@gmail.com>
> Fixes: c6e67f5a9373e916a8d2 "gallium/swr: add OpenSWR rasterizer"
> Reviewed-by: Eric Engestrom <eric.engest...@imgtec.com>
> Signed-off-by: Emil Velikov <emil.veli...@collabora.com>
> ---
> src/gallium/drivers/swr/Makefile.am | 8 ++++++++
> 1 file changed, 8 insertions(+)
> 
> diff --git a/src/gallium/drivers/swr/Makefile.am 
> b/src/gallium/drivers/swr/Makefile.am
> index 64950214572..02010727d9b 100644
> --- a/src/gallium/drivers/swr/Makefile.am
> +++ b/src/gallium/drivers/swr/Makefile.am
> @@ -246,6 +246,7 @@ if HAVE_SWR_AVX
> lib_LTLIBRARIES += libswrAVX.la
> 
> libswrAVX_la_CXXFLAGS = \
> +     $(PTHREAD_CFLAGS) \
>       $(SWR_AVX_CXXFLAGS) \
>       -DKNOB_ARCH=KNOB_ARCH_AVX \
>       $(COMMON_CXXFLAGS)
> @@ -253,6 +254,9 @@ libswrAVX_la_CXXFLAGS = \
> libswrAVX_la_SOURCES = \
>       $(COMMON_SOURCES)
> 
> +libswrAVX_la_LIBADD = \
> +     $(PTHREAD_LIBS)
> +
> libswrAVX_la_LDFLAGS = \
>       $(COMMON_LDFLAGS)
> endif
> @@ -260,6 +264,7 @@ endif
> if HAVE_SWR_AVX2
> lib_LTLIBRARIES += libswrAVX2.la
> libswrAVX2_la_CXXFLAGS = \
> +     $(PTHREAD_CFLAGS) \
>       $(SWR_AVX2_CXXFLAGS) \
>       -DKNOB_ARCH=KNOB_ARCH_AVX2 \
>       $(COMMON_CXXFLAGS)
> @@ -267,6 +272,9 @@ libswrAVX2_la_CXXFLAGS = \
> libswrAVX2_la_SOURCES = \
>       $(COMMON_SOURCES)
> 
> +libswrAVX2_la_LIBADD = \
> +     $(PTHREAD_LIBS)
> +
> libswrAVX2_la_LDFLAGS = \
>       $(COMMON_LDFLAGS)
> endif
> -- 
> 2.13.0
> 

_______________________________________________
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/mesa-dev

Reply via email to