Mesa (master): r600g: Don't build llvm_wrapper.cpp when we aren' t using LLVM

Tom Stellard tstellar at kemper.freedesktop.org
Fri Jan 25 21:59:04 UTC 2013


Module: Mesa
Branch: master
Commit: 264e6dad28e64755dc1580abdbb4e339c3439883
URL:    http://cgit.freedesktop.org/mesa/mesa/commit/?id=264e6dad28e64755dc1580abdbb4e339c3439883

Author: Tom Stellard <thomas.stellard at amd.com>
Date:   Fri Jan 25 15:01:47 2013 +0000

r600g: Don't build llvm_wrapper.cpp when we aren't using LLVM

We were using the NEED_RADEON_GALLIUM conditional to decide whether or not
to build llvm_wrapper.cpp, which is required for using the LLVM backend.
llvm_wrapper.cpp needs to be linked against the LLVM IPO libary
and this library is only added to LLVM_LIBS if either opencl or the
r600-llvm-compiler is enabled.

The NEED_RADEON_GALLIUM conditional is set to true when enabling the
radeonsi driver, so if the radeonsi and r600 drivers are enabled without
also enabling opencl or r600-llvm-compiler, llvm_wrapper.cpp will be
built, but the IPO library won't be added to LLVM_LIBS.  This was
causing unresolved symbol errors when buiding with this configuration.

https://bugs.freedesktop.org/show_bug.cgi?id=59831

Tested-by: Alex Deucher <alexander.deucher at amd.com>

---

 src/gallium/drivers/r600/Makefile.am |    4 +++-
 1 files changed, 3 insertions(+), 1 deletions(-)

diff --git a/src/gallium/drivers/r600/Makefile.am b/src/gallium/drivers/r600/Makefile.am
index 995261b..6de7e0f 100644
--- a/src/gallium/drivers/r600/Makefile.am
+++ b/src/gallium/drivers/r600/Makefile.am
@@ -13,7 +13,8 @@ AM_CFLAGS = \
 libr600_la_SOURCES = \
 	$(C_SOURCES)
 
-if NEED_RADEON_GALLIUM
+if USE_R600_LLVM_COMPILER
+if HAVE_GALLIUM_COMPUTE
 
 libr600_la_SOURCES += \
 	$(LLVM_C_SOURCES) \
@@ -28,6 +29,7 @@ AM_CFLAGS += \
 AM_CXXFLAGS= \
 	$(LLVM_CXXFLAGS)
 endif
+endif
 
 if USE_R600_LLVM_COMPILER
 AM_CFLAGS += \




More information about the mesa-commit mailing list