[Mesa-dev] [PATCH 4/4] configure.ac: Don't check LLVM version in gallium_require_llvm

Tobias Droste tdroste at gmx.de
Sat Jan 28 13:57:00 UTC 2017


This is actually not needed because the version is checked later.

Line 2403:
if test "x$enable_gallium_llvm" == "xyes"; then
    llvm_require_version $LLVM_REQUIRED_GALLIUM "gallium"
    llvm_add_default_components "gallium"

    USE_LLVM_GALLIUM=yes
    DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT -DMESA_LLVM_VERSION_PATCH=$LLVM_VERSION_PATCH"
else
    USE_LLVM_GALLIUM=no
fi

Signed-off-by: Tobias Droste <tdroste at gmx.de>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com>
---
 configure.ac | 4 +---
 1 file changed, 1 insertion(+), 3 deletions(-)

diff --git a/configure.ac b/configure.ac
index 5101e3bb0b..b1fb2c3c54 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2208,9 +2208,7 @@ dnl
 dnl Gallium helper functions
 dnl
 gallium_require_llvm() {
-    if test "x$enable_gallium_llvm" = "xyes"; then
-        llvm_require_version $LLVM_REQUIRED_GALLIUM "gallium"
-    else
+    if test "x$enable_gallium_llvm" != "xyes"; then
         AC_MSG_ERROR([--enable-gallium-llvm is required when building $1])
     fi
 }
-- 
2.11.0



More information about the mesa-dev mailing list