Hi Vinson, this should not be needed and is obsolete as soon as this gets merged (Emil?): https://lists.freedesktop.org/archives/mesa-dev/2016-November/135840.html
Did you actually need this or were you just seeing that the code is wrong? Because LLVM >=3.1 has the header files and this code should never execute! Tobias Am Samstag, 3. Dezember 2016, 01:20:42 CET schrieb Vinson Lee: > HAVE_LLVM variable included the patch version if the LLVM version had a > patch version. > > For LLVM version '4.0.0', HAVE_LLVM would be '0x0400.0'. > > Fixes: 45574ab2e92f ("configure.ac: better detection of LLVM version") > Signed-off-by: Vinson Lee <v...@freedesktop.org> > --- > configure.ac | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/configure.ac b/configure.ac > index f62bc61e5025..3b8b32485ae1 100644 > --- a/configure.ac > +++ b/configure.ac > @@ -2195,7 +2195,7 @@ if test "x$enable_gallium_llvm" = xyes || test > "x$HAVE_RADEON_VULKAN" = xyes; th if test -n "${LLVM_VERSION_MAJOR}"; then > LLVM_VERSION_INT="${LLVM_VERSION_MAJOR}0${LLVM_VERSION_MINOR}" > else > - LLVM_VERSION_INT=`echo $LLVM_VERSION | sed -e > 's/\([[0-9]]\)\.\([[0-9]]\)/\10\2/g'` + LLVM_VERSION_INT=`echo > $LLVM_VERSION | sed -e 's/\([[0-9]]\)\.\([[0-9]]\).*/\10\2/g'` fi > > LLVM_REQUIRED_VERSION_MAJOR="3" _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev