[Mesa-stable] [PATCH 1/3] swr: don't forget to link AVX/AVX2 against pthreads

Rowley, Timothy O timothy.o.rowley at intel.com
Fri Jul 21 19:51:49 UTC 2017


Reviewed-by: Tim Rowley <timothy.o.rowley at intel.com>

> On Jul 21, 2017, at 1:05 PM, Emil Velikov <emil.l.velikov at gmail.com> wrote:
> 
> From: Emil Velikov <emil.velikov at 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-stable at lists.freedesktop.org
> Cc: Bruce Cherniak <bruce.cherniak at intel.com>
> Cc: Tim Rowley <timothy.o.rowley at intel.com>
> Cc: Laurent Carlier <lordheavym at gmail.com>
> Fixes: c6e67f5a9373e916a8d2 "gallium/swr: add OpenSWR rasterizer"
> Reviewed-by: Eric Engestrom <eric.engestrom at imgtec.com>
> Signed-off-by: Emil Velikov <emil.velikov at 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
> 



More information about the mesa-stable mailing list