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