From: Juan A. Suarez Romero Date: Mon, 18 Jun 2018 13:45:51 +0000 (+0200) Subject: swr: bump minimum supported LLVM version to 5.0 X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=d24839be70d66cb74f4eb3c5f76b7ed634bdc7d3;p=mesa.git swr: bump minimum supported LLVM version to 5.0 RADV now requires LLVM 5.0 or greater, and thus we can't build dist tarball because swr requires LLVM 4.0. Let's bump required LLVM to 5.0 in swr too. Fixes: f9eb1ef870 ("amd: remove support for LLVM 4.0") Cc: Tim Rowley Cc: Emil Velikov Cc: Dylan Baker Cc: Eric Engestrom Reviewed-by: Eric Engestrom Reviewed-by: Emil Velikov Acked-by: Bruce Cherniak --- diff --git a/.travis.yml b/.travis.yml index b1fc7de9587..c9a30fa0ef5 100644 --- a/.travis.yml +++ b/.travis.yml @@ -92,7 +92,7 @@ matrix: - BUILD=make - MAKEFLAGS="-j4" - MAKE_CHECK_COMMAND="true" - - LLVM_VERSION=4.0 + - LLVM_VERSION=5.0 - LLVM_CONFIG="llvm-config-${LLVM_VERSION}" - OVERRIDE_CC="gcc-4.8" - OVERRIDE_CXX="g++-4.8" @@ -105,12 +105,12 @@ matrix: addons: apt: sources: - - llvm-toolchain-trusty-4.0 + - llvm-toolchain-trusty-5.0 packages: # LLVM packaging is broken and misses these dependencies - libedit-dev # From sources above - - llvm-4.0-dev + - llvm-5.0-dev # Common - xz-utils - x11proto-xf86vidmode-dev @@ -432,7 +432,7 @@ matrix: - BUILD=scons - SCONSFLAGS="-j4" - SCONS_TARGET="swr=1" - - LLVM_VERSION=4.0 + - LLVM_VERSION=5.0 - LLVM_CONFIG="llvm-config-${LLVM_VERSION}" # Keep it symmetrical to the make build. There's no actual SWR, yet. - SCONS_CHECK_COMMAND="true" @@ -441,13 +441,13 @@ matrix: addons: apt: sources: - - llvm-toolchain-trusty-4.0 + - llvm-toolchain-trusty-5.0 packages: - scons # LLVM packaging is broken and misses these dependencies - libedit-dev # From sources above - - llvm-4.0-dev + - llvm-5.0-dev # Common - xz-utils - x11proto-xf86vidmode-dev diff --git a/configure.ac b/configure.ac index 7af5d9a0e44..a1953620071 100644 --- a/configure.ac +++ b/configure.ac @@ -110,7 +110,7 @@ LLVM_REQUIRED_OPENCL=3.9.0 LLVM_REQUIRED_R600=3.9.0 LLVM_REQUIRED_RADEONSI=5.0.0 LLVM_REQUIRED_RADV=5.0.0 -LLVM_REQUIRED_SWR=4.0.0 +LLVM_REQUIRED_SWR=5.0.0 dnl Check for progs AC_PROG_CPP @@ -2793,8 +2793,9 @@ if test -n "$with_gallium_drivers"; then fi # XXX: Keep in sync with LLVM_REQUIRED_SWR -AM_CONDITIONAL(SWR_INVALID_LLVM_VERSION, test "x$LLVM_VERSION" != x4.0.0 -a \ - "x$LLVM_VERSION" != x4.0.1) +AM_CONDITIONAL(SWR_INVALID_LLVM_VERSION, test "x$LLVM_VERSION" != x5.0.0 -a \ + "x$LLVM_VERSION" != x5.0.1 -a \ + "x$LLVM_VERSION" != x5.0.2) if test "x$enable_llvm" = "xyes" -a "$with_gallium_drivers"; then llvm_require_version $LLVM_REQUIRED_GALLIUM "gallium" diff --git a/meson.build b/meson.build index 76aa10f81a5..a2d59776c05 100644 --- a/meson.build +++ b/meson.build @@ -1144,10 +1144,8 @@ if with_gallium_opencl llvm_optional_modules += ['coroutines', 'opencl'] endif -if with_amd_vk or with_gallium_radeonsi +if with_amd_vk or with_gallium_radeonsi or with_gallium_swr _llvm_version = '>= 5.0.0' -elif with_gallium_swr - _llvm_version = '>= 4.0.0' elif with_gallium_opencl or with_gallium_r600 _llvm_version = '>= 3.9.0' else diff --git a/src/gallium/drivers/swr/Makefile.am b/src/gallium/drivers/swr/Makefile.am index 8b3150288e6..5cc3f77478a 100644 --- a/src/gallium/drivers/swr/Makefile.am +++ b/src/gallium/drivers/swr/Makefile.am @@ -374,9 +374,9 @@ include $(top_srcdir)/install-gallium-links.mk # created with the oldest supported version of LLVM. dist-hook: if SWR_INVALID_LLVM_VERSION - @echo "*******************************************************" - @echo "LLVM 4.0.0 or LLVM 4.0.1 required to create the tarball" - @echo "*******************************************************" + @echo "*****************************************" + @echo "LLVM 5.0.x required to create the tarball" + @echo "*****************************************" @test endif diff --git a/src/gallium/drivers/swr/SConscript b/src/gallium/drivers/swr/SConscript index 528cfac39f6..224372eb3f5 100644 --- a/src/gallium/drivers/swr/SConscript +++ b/src/gallium/drivers/swr/SConscript @@ -12,8 +12,8 @@ if not env['llvm']: env['swr'] = False Return() -if env['LLVM_VERSION'] < distutils.version.LooseVersion('4.0'): - print("warning: swr requires LLVM >= 4.0: not building swr") +if env['LLVM_VERSION'] < distutils.version.LooseVersion('5.0'): + print("warning: swr requires LLVM >= 5.0: not building swr") env['swr'] = False Return()