LLVM components are only added when LLVM is needed.
This means gallium adds this as soon as "--enable-gallium-llvm"
is "yes" and radv + opencl add it explicitly.

This slightly changes the "auto" value of "--enable-gallium-llvm":
If no LLVM is installed, it is set to "no". If later drivers need
this they get the error message that "--enable-gallium-llvm" is
needed.
This way if someone is only building softpipe he doesn't need to
specify "--disable-gallium-llvm" and doesn't have to have LLVM
installed.

Signed-off-by: Tobias Droste <tdro...@gmx.de>
---
 configure.ac | 21 +++++++++++++++++----
 1 file changed, 17 insertions(+), 4 deletions(-)

diff --git a/configure.ac b/configure.ac
index bbc1cb5..9e7130ad 100644
--- a/configure.ac
+++ b/configure.ac
@@ -897,7 +897,11 @@ llvm_add_component() {
     new_llvm_component=$1
     driver_name=$2
 
-    LLVM_COMPONENTS="$LLVM_COMPONENTS $new_llvm_component"
+    if $LLVM_CONFIG --components | grep -iqw $new_llvm_component ; then
+        LLVM_COMPONENTS="${LLVM_COMPONENTS} ${new_llvm_component}"
+    else
+        AC_MSG_ERROR([LLVM component '$new_llvm_component' not enabled in your 
LLVM build. Required by $driver_name.])
+    fi
 }
 
 llvm_add_default_components() {
@@ -980,8 +984,6 @@ llvm_set_environment_variables() {
             LLVM_VERSION_INT=`echo $LLVM_VERSION | sed -e 
's/\([[0-9]]\)\.\([[0-9]]\)/\10\2/g'`
         fi
 
-        llvm_add_default_components "gallium"
-
         DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT 
-DMESA_LLVM_VERSION_PATCH=$LLVM_VERSION_PATCH"
         MESA_LLVM=1
     else
@@ -1007,6 +1009,7 @@ radeon_llvm_check() {
 
     llvm_add_target $amdgpu_llvm_target_name $4
 
+    llvm_add_default_components $4
     llvm_add_component "bitreader" $4
     llvm_add_component "ipo" $4
 
@@ -2175,6 +2178,7 @@ if test "x$enable_opencl" = xyes; then
 
     llvm_check_version_for "3" "6" "0" "opencl"
 
+    llvm_add_default_components "opencl"
     llvm_add_component "all-targets" "opencl"
     llvm_add_component "linker" "opencl"
     llvm_add_component "instrumentation" "opencl"
@@ -2322,9 +2326,18 @@ if test -z "$with_gallium_drivers"; then
 fi
 if test "x$enable_gallium_llvm" = xauto; then
     case "$host_cpu" in
-    i*86|x86_64|amd64) enable_gallium_llvm=yes;;
+    i*86|x86_64|amd64)
+        if test "x$MESA_LLVM" = "x1"; then
+            enable_gallium_llvm=yes
+        else
+            enable_gallium_llvm=no
+        fi
+        ;;
     esac
 fi
+if test "x$enable_gallium_llvm" == "xyes"; then
+    llvm_add_default_components "gallium"
+fi
 
 dnl Directory for XVMC libs
 AC_ARG_WITH([xvmc-libdir],
-- 
2.10.1

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

Reply via email to