[Mesa-dev] [PATCH 03/17] configure.ac: Don't check LLVM version in require_llvm

Emil Velikov emil.l.velikov at gmail.com
Tue Feb 7 22:44:45 UTC 2017


From: Tobias Droste <tdroste at gmx.de>

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

Around line 2380
if test "x$enable_gallium_llvm" == "xyes"; then
    llvm_check_version_for $LLVM_REQUIRED_GALLIUM "gallium"
    llvm_add_default_components "gallium"
fi

Cc: "17.0" <mesa-stable at lists.freedesktop.org>
Cc: Tobias Droste <tdroste at gmx.de>
Signed-off-by: Tobias Droste <tdroste at gmx.de>
Reviewed-by: Emil Velikov <emil.velikov at collabora.com> (v1)
v2: [Emil Velikov: rebase/respin series order]
Signed-off-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 0418421d5a..0c15c3f51d 100644
--- a/configure.ac
+++ b/configure.ac
@@ -2192,9 +2192,7 @@ dnl
 dnl Gallium helper functions
 dnl
 gallium_require_llvm() {
-    if test "x$enable_gallium_llvm" = "xyes"; then
-        llvm_check_version_for $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