From 0484b8446a02a4b77d5a1dab89f6e0b8af831027 Mon Sep 17 00:00:00 2001 From: Emil Velikov Date: Tue, 11 Mar 2014 15:24:07 +0000 Subject: [PATCH] gallium/targets: explicitly include a dummy.cpp and remove all the LINK mayhem Explicitly setting the linker variable was required for old and broken build toolchains. At this point this should no longer be needed, and setting the sources lists will trigger generation of the correct LINK variables. Explicitly include dummy.cpp to use g++ to link the static library which in most cases is based upon C++ code. v2: Reword commit message. Signed-off-by: Emil Velikov Reviewed-by: Matt Turner --- src/gallium/targets/dri-i915/Makefile.am | 3 +-- src/gallium/targets/dri-ilo/Makefile.am | 4 +--- src/gallium/targets/dri-swrast/Makefile.am | 3 +-- src/gallium/targets/dri-vmwgfx/Makefile.am | 5 +---- src/gallium/targets/gbm/Makefile.am | 10 +--------- src/gallium/targets/libgl-xlib/Makefile.am | 9 +-------- src/gallium/targets/opencl/Makefile.am | 3 +-- src/gallium/targets/osmesa/Makefile.am | 4 +--- src/gallium/targets/r600/dri/Makefile.am | 3 +-- src/gallium/targets/r600/omx/Makefile.am | 12 +----------- src/gallium/targets/r600/vdpau/Makefile.am | 5 +---- src/gallium/targets/r600/xvmc/Makefile.am | 12 +----------- src/gallium/targets/radeonsi/omx/Makefile.am | 8 +------- src/gallium/targets/xa/Makefile.am | 8 +------- 14 files changed, 14 insertions(+), 75 deletions(-) diff --git a/src/gallium/targets/dri-i915/Makefile.am b/src/gallium/targets/dri-i915/Makefile.am index c9988e3d1a3..d51144da3b7 100644 --- a/src/gallium/targets/dri-i915/Makefile.am +++ b/src/gallium/targets/dri-i915/Makefile.am @@ -33,6 +33,7 @@ AM_CPPFLAGS = \ dridir = $(DRI_DRIVER_INSTALL_DIR) dri_LTLIBRARIES = i915_dri.la +nodist_EXTRA_i915_dri_la_SOURCES = dummy.cpp i915_dri_la_SOURCES = target.c i915_dri_la_LDFLAGS = $(GALLIUM_DRI_LINKER_FLAGS) @@ -52,8 +53,6 @@ i915_dri_la_LIBADD = \ $(GALLIUM_DRI_LIB_DEPS) \ $(INTEL_LIBS) -nodist_EXTRA_i915_dri_la_SOURCES = dummy.cpp - if HAVE_MESA_LLVM AM_CPPFLAGS += -DGALLIUM_LLVMPIPE i915_dri_la_LIBADD += $(top_builddir)/src/gallium/drivers/llvmpipe/libllvmpipe.la diff --git a/src/gallium/targets/dri-ilo/Makefile.am b/src/gallium/targets/dri-ilo/Makefile.am index f04119ba801..d38dcdc3d00 100644 --- a/src/gallium/targets/dri-ilo/Makefile.am +++ b/src/gallium/targets/dri-ilo/Makefile.am @@ -33,6 +33,7 @@ AM_CPPFLAGS = \ dridir = $(DRI_DRIVER_INSTALL_DIR) dri_LTLIBRARIES = ilo_dri.la +nodist_EXTRA_ilo_dri_la_SOURCES = dummy.cpp ilo_dri_la_SOURCES = target.c ilo_dri_la_LDFLAGS = $(GALLIUM_DRI_LINKER_FLAGS) @@ -50,7 +51,4 @@ ilo_dri_la_LIBADD = \ $(GALLIUM_DRI_LIB_DEPS) \ $(INTEL_LIBS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_ilo_dri_la_SOURCES = dummy-cpp.cpp - include $(top_srcdir)/install-gallium-links.mk diff --git a/src/gallium/targets/dri-swrast/Makefile.am b/src/gallium/targets/dri-swrast/Makefile.am index 06eb16aec49..5238459609a 100644 --- a/src/gallium/targets/dri-swrast/Makefile.am +++ b/src/gallium/targets/dri-swrast/Makefile.am @@ -36,6 +36,7 @@ AM_CPPFLAGS = \ dridir = $(DRI_DRIVER_INSTALL_DIR) dri_LTLIBRARIES = swrast_dri.la +nodist_EXTRA_swrast_dri_la_SOURCES = dummy.cpp swrast_dri_la_SOURCES = \ swrast_drm_api.c \ $(top_srcdir)/src/mesa/drivers/dri/common/utils.c \ @@ -54,8 +55,6 @@ swrast_dri_la_LIBADD = \ $(top_builddir)/src/gallium/drivers/rbug/librbug.la \ $(GALLIUM_DRI_LIB_DEPS) -nodist_EXTRA_swrast_dri_la_SOURCES = dummy.cpp - if HAVE_MESA_LLVM AM_CPPFLAGS += -DGALLIUM_LLVMPIPE swrast_dri_la_LIBADD += $(top_builddir)/src/gallium/drivers/llvmpipe/libllvmpipe.la diff --git a/src/gallium/targets/dri-vmwgfx/Makefile.am b/src/gallium/targets/dri-vmwgfx/Makefile.am index 871dc0d6a26..ee5f07bee5e 100644 --- a/src/gallium/targets/dri-vmwgfx/Makefile.am +++ b/src/gallium/targets/dri-vmwgfx/Makefile.am @@ -31,6 +31,7 @@ AM_CPPFLAGS = \ dridir = $(DRI_DRIVER_INSTALL_DIR) dri_LTLIBRARIES = vmwgfx_dri.la +nodist_EXTRA_vmwgfx_dri_la_SOURCES = dummy.cpp vmwgfx_dri_la_SOURCES = \ target.c \ vmw_powf.c @@ -49,8 +50,4 @@ vmwgfx_dri_la_LIBADD = \ $(top_builddir)/src/gallium/drivers/svga/libsvga.la \ $(GALLIUM_DRI_LIB_DEPS) -vmwgfx_dri_la_LINK = $(CXXLINK) $(vmwgfx_dri_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_vmwgfx_dri_la_SOURCES = dummy-cpp.cpp - include $(top_srcdir)/install-gallium-links.mk diff --git a/src/gallium/targets/gbm/Makefile.am b/src/gallium/targets/gbm/Makefile.am index d5255e94b86..e36d317b069 100644 --- a/src/gallium/targets/gbm/Makefile.am +++ b/src/gallium/targets/gbm/Makefile.am @@ -39,6 +39,7 @@ AM_CFLAGS = \ gbm_LTLIBRARIES = gbm_gallium_drm.la +nodist_EXTRA_gbm_gallium_drm_la_SOURCES = dummy.cpp gbm_gallium_drm_la_SOURCES = gbm.c gbm_gallium_drm_la_LIBADD = \ @@ -64,18 +65,9 @@ endif gbm_gallium_drm_la_LDFLAGS = -no-undefined -avoid-version -module -# FIXME: this shouldn't be needed if HAVE_MESA_LLVM -gbm_gallium_drm_la_LINK = $(CXXLINK) $(gbm_gallium_drm_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_gbm_gallium_drm_la_SOURCES = dummy-cpp.cpp - gbm_gallium_drm_la_LIBADD += $(LLVM_LIBS) gbm_gallium_drm_la_LDFLAGS += $(LLVM_LDFLAGS) -else -gbm_gallium_drm_la_LINK = $(CXXLINK) $(gbm_gallium_drm_la_LDFLAGS) -# Mention a dummy pure C file to trigger generation of the $(LINK) variable -nodist_EXTRA_gbm_gallium_drm_la_SOURCES = dummy-c.c endif TESTS = gallium-gbm-symbols-check diff --git a/src/gallium/targets/libgl-xlib/Makefile.am b/src/gallium/targets/libgl-xlib/Makefile.am index 7e84ffe1262..cc382db08b2 100644 --- a/src/gallium/targets/libgl-xlib/Makefile.am +++ b/src/gallium/targets/libgl-xlib/Makefile.am @@ -42,6 +42,7 @@ AM_CFLAGS = $(X11_INCLUDES) lib_LTLIBRARIES = libGL.la +nodist_EXTRA_libGL_la_SOURCES = dummy.cpp libGL_la_SOURCES = xlib.c libGL_la_LDFLAGS = -version-number $(GL_MAJOR):$(GL_MINOR):$(GL_TINY) -no-undefined @@ -59,17 +60,9 @@ libGL_la_LIBADD = \ $(CLOCK_LIB) if HAVE_MESA_LLVM -libGL_la_LINK = $(CXXLINK) $(libGL_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_libGL_la_SOURCES = dummy-cpp.cpp - libGL_la_LIBADD += $(top_builddir)/src/gallium/drivers/llvmpipe/libllvmpipe.la $(LLVM_LIBS) AM_CPPFLAGS += -DGALLIUM_LLVMPIPE libGL_la_LDFLAGS += $(LLVM_LDFLAGS) -else -libGL_la_LINK = $(CXXLINK) $(libGL_la_LDFLAGS) -# Mention a dummy pure C file to trigger generation of the $(LINK) variable -nodist_EXTRA_libGL_la_SOURCES = dummy-c.c endif include $(top_srcdir)/install-gallium-links.mk diff --git a/src/gallium/targets/opencl/Makefile.am b/src/gallium/targets/opencl/Makefile.am index 20a52476842..eb25a5e47aa 100644 --- a/src/gallium/targets/opencl/Makefile.am +++ b/src/gallium/targets/opencl/Makefile.am @@ -41,10 +41,9 @@ lib@OPENCL_LIBNAME@_la_LIBADD += \ $(LIBDRM_LIBS) endif +nodist_EXTRA_lib@OPENCL_LIBNAME@_la_SOURCES = dummy.cpp lib@OPENCL_LIBNAME@_la_SOURCES = -# Force usage of a C++ linker -nodist_EXTRA_lib@OPENCL_LIBNAME@_la_SOURCES = dummy.cpp if HAVE_CLOVER_ICD icddir = /etc/OpenCL/vendors/ diff --git a/src/gallium/targets/osmesa/Makefile.am b/src/gallium/targets/osmesa/Makefile.am index 48154e1c2d3..2b133c44cb9 100644 --- a/src/gallium/targets/osmesa/Makefile.am +++ b/src/gallium/targets/osmesa/Makefile.am @@ -36,6 +36,7 @@ AM_CPPFLAGS = \ lib_LTLIBRARIES = lib@OSMESA_LIB@.la +nodist_EXTRA_lib@OSMESA_LIB@_la_SOURCES = dummy.cpp lib@OSMESA_LIB@_la_SOURCES = target.c lib@OSMESA_LIB@_la_LDFLAGS = -module -version-number @OSMESA_VERSION@ -no-undefined @@ -56,9 +57,6 @@ lib@OSMESA_LIB@_la_LIBADD = \ $(OSMESA_LIB_DEPS) \ $(CLOCK_LIB) - -nodist_EXTRA_lib@OSMESA_LIB@_la_SOURCES = dummy.cpp - if HAVE_MESA_LLVM AM_CPPFLAGS += -DGALLIUM_LLVMPIPE lib@OSMESA_LIB@_la_LDFLAGS += $(LLVM_LDFLAGS) diff --git a/src/gallium/targets/r600/dri/Makefile.am b/src/gallium/targets/r600/dri/Makefile.am index 9f7c096ddf7..f828a734379 100644 --- a/src/gallium/targets/r600/dri/Makefile.am +++ b/src/gallium/targets/r600/dri/Makefile.am @@ -33,6 +33,7 @@ AM_CPPFLAGS = \ dridir = $(DRI_DRIVER_INSTALL_DIR) dri_LTLIBRARIES = r600_dri.la +nodist_EXTRA_r600_dri_la_SOURCES = dummy.cpp r600_dri_la_SOURCES = \ drm_target.c @@ -54,6 +55,4 @@ r600_dri_la_LIBADD = \ $(LIBDRM_LIBS) \ $(RADEON_LIBS) -nodist_EXTRA_r600_dri_la_SOURCES = dummy.cpp - include $(top_srcdir)/install-gallium-links.mk diff --git a/src/gallium/targets/r600/omx/Makefile.am b/src/gallium/targets/r600/omx/Makefile.am index 1f07595d882..22ef08f3a41 100644 --- a/src/gallium/targets/r600/omx/Makefile.am +++ b/src/gallium/targets/r600/omx/Makefile.am @@ -36,6 +36,7 @@ AM_CPPFLAGS = \ omxdir = $(OMX_LIB_INSTALL_DIR) omx_LTLIBRARIES = libomx_r600.la +nodist_EXTRA_libomx_r600_la_SOURCES = dummy.cpp libomx_r600_la_SOURCES = \ drm_target.c \ $(top_srcdir)/src/gallium/auxiliary/vl/vl_winsys_dri.c @@ -50,14 +51,3 @@ libomx_r600_la_LIBADD = \ $(LIBDRM_LIBS) \ $(RADEON_LIBS) \ -lstdc++ - -if HAVE_MESA_LLVM -libomx_r600_la_LINK = $(CXXLINK) $(libomx_r600_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_libomx_r600_la_SOURCES = dummy-cpp.cpp - -else -libomx_r600_la_LINK = $(LINK) $(libomx_r600_la_LDFLAGS) -# Mention a dummy pure C file to trigger generation of the $(LINK) variable -nodist_EXTRA_libomx_r600_la_SOURCES = dummy-c.c -endif diff --git a/src/gallium/targets/r600/vdpau/Makefile.am b/src/gallium/targets/r600/vdpau/Makefile.am index 1ff35e0d296..455d90f795b 100644 --- a/src/gallium/targets/r600/vdpau/Makefile.am +++ b/src/gallium/targets/r600/vdpau/Makefile.am @@ -30,6 +30,7 @@ AM_CFLAGS = \ vdpaudir = $(VDPAU_LIB_INSTALL_DIR) vdpau_LTLIBRARIES = libvdpau_r600.la +nodist_EXTRA_libvdpau_r600_la_SOURCES = dummy.cpp libvdpau_r600_la_SOURCES = \ drm_target.c \ $(top_srcdir)/src/gallium/auxiliary/vl/vl_winsys_dri.c @@ -44,8 +45,4 @@ libvdpau_r600_la_LIBADD = \ $(GALLIUM_VDPAU_LIB_DEPS) \ $(RADEON_LIBS) -libvdpau_r600_la_LINK = $(CXXLINK) $(libvdpau_r600_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_libvdpau_r600_la_SOURCES = dummy-cpp.cpp - include $(top_srcdir)/install-gallium-links.mk diff --git a/src/gallium/targets/r600/xvmc/Makefile.am b/src/gallium/targets/r600/xvmc/Makefile.am index 55775af72b1..56b0a6d75bf 100644 --- a/src/gallium/targets/r600/xvmc/Makefile.am +++ b/src/gallium/targets/r600/xvmc/Makefile.am @@ -28,6 +28,7 @@ AM_CFLAGS = \ xvmcdir = $(XVMC_LIB_INSTALL_DIR) xvmc_LTLIBRARIES = libXvMCr600.la +nodist_EXTRA_libXvMCr600_la_SOURCES = dummy.cpp libXvMCr600_la_SOURCES = \ drm_target.c \ $(top_srcdir)/src/gallium/auxiliary/vl/vl_winsys_dri.c @@ -41,14 +42,3 @@ libXvMCr600_la_LIBADD = \ $(GALLIUM_XVMC_LIB_DEPS) \ $(GALLIUM_DRI_LIB_DEPS) \ $(RADEON_LIBS) - -if HAVE_MESA_LLVM -libXvMCr600_la_LINK = $(CXXLINK) $(libXvMCr600_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_libXvMCr600_la_SOURCES = dummy-cpp.cpp - -else -libXvMCr600_la_LINK = $(LINK) $(libXvMCr600_la_LDFLAGS) -# Mention a dummy pure C file to trigger generation of the $(LINK) variable -nodist_EXTRA_libXvMCr600_la_SOURCES = dummy-c.c -endif diff --git a/src/gallium/targets/radeonsi/omx/Makefile.am b/src/gallium/targets/radeonsi/omx/Makefile.am index c2065cf8926..439e91c0578 100644 --- a/src/gallium/targets/radeonsi/omx/Makefile.am +++ b/src/gallium/targets/radeonsi/omx/Makefile.am @@ -36,6 +36,7 @@ AM_CPPFLAGS = \ omxdir = $(OMX_LIB_INSTALL_DIR) omx_LTLIBRARIES = libomx_radeonsi.la +nodist_EXTRA_libomx_radeonsi_la_SOURCES = dummy.cpp libomx_radeonsi_la_SOURCES = \ drm_target.c \ $(top_srcdir)/src/gallium/auxiliary/vl/vl_winsys_dri.c @@ -49,10 +50,3 @@ libomx_radeonsi_la_LIBADD = \ $(GALLIUM_OMX_LIB_DEPS) \ $(LIBDRM_LIBS) \ $(RADEON_LIBS) - -if HAVE_MESA_LLVM -libomx_radeonsi_la_LINK = $(CXXLINK) $(libomx_radeonsi_la_LDFLAGS) -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_libomx_radeonsi_la_SOURCES = dummy-cpp.cpp - -endif diff --git a/src/gallium/targets/xa/Makefile.am b/src/gallium/targets/xa/Makefile.am index 1753f1d49fa..1d4f095da33 100644 --- a/src/gallium/targets/xa/Makefile.am +++ b/src/gallium/targets/xa/Makefile.am @@ -36,6 +36,7 @@ pkgconfig_DATA = xatracker.pc lib_LTLIBRARIES = libxatracker.la +nodist_EXTRA_libxatracker_la_SOURCES = dummy.cpp libxatracker_la_SOURCES = libxatracker_la_LIBADD = \ @@ -62,16 +63,9 @@ libxatracker_la_LDFLAGS = \ -no-undefined \ -version-number $(XA_MAJOR):$(XA_MINOR):$(XA_TINY) -# FIXME: this shouldn't be needed if HAVE_MESA_LLVM -# Mention a dummy pure C++ file to trigger generation of the $(LINK) variable -nodist_EXTRA_libxatracker_la_SOURCES = dummy-cpp.cpp - libxatracker_la_LIBADD += $(LLVM_LIBS) libxatracker_la_LDFLAGS += $(LLVM_LDFLAGS) -else -# Mention a dummy pure C file to trigger generation of the $(LINK) variable -nodist_EXTRA_libxatracker_la_SOURCES = dummy-c.c endif include $(top_srcdir)/install-lib-links.mk -- 2.30.2