[Mesa-dev] [Bug 66029] More robust way of detecting LLVM major and minor versions

2013-07-19 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=66029

Tom Stellard  changed:

   What|Removed |Added

 Status|NEW |RESOLVED
 Resolution|--- |FIXED

--- Comment #5 from Tom Stellard  ---
Committed as 45574ab2e92f0bf74b18448baff49cb2eb5db620

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [Bug 66029] More robust way of detecting LLVM major and minor versions

2013-06-28 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=66029

--- Comment #4 from Klemens Baum  ---
Sent: http://lists.freedesktop.org/archives/mesa-dev/2013-June/041160.html

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [Bug 66029] More robust way of detecting LLVM major and minor versions

2013-06-27 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=66029

--- Comment #3 from Tom Stellard  ---
Hi, this patch looks good at first glance.  Could you generate the patch using
git format-patch and send it to the mailing list
(mesa-dev@lists.freedesktop.org)

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [Bug 66029] More robust way of detecting LLVM major and minor versions

2013-06-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=66029

Klemens Baum  changed:

   What|Removed |Added

  Attachment #81187|0   |1
is obsolete||

--- Comment #2 from Klemens Baum  ---
Created attachment 81188
  --> https://bugs.freedesktop.org/attachment.cgi?id=81188&action=edit
Use the new version defines if available

Sorry for the previous comment, didn't mark as patch.

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev


[Mesa-dev] [Bug 66029] More robust way of detecting LLVM major and minor versions

2013-06-21 Thread bugzilla-daemon
https://bugs.freedesktop.org/show_bug.cgi?id=66029

Klemens Baum  changed:

   What|Removed |Added

  Attachment #81187|0   |1
   is patch||

--- Comment #1 from Klemens Baum  ---
Comment on attachment 81187
  --> https://bugs.freedesktop.org/attachment.cgi?id=81187
Use the new version defines if available

>diff --git a/configure.ac b/configure.ac
>index 6832b0d..3cabfe8 100644
>--- a/configure.ac
>+++ b/configure.ac
>@@ -1577,8 +1577,26 @@ if test "x$enable_gallium_llvm" = xyes; then
> fi
>
> if test "x$LLVM_CONFIG" != xno; then
>-  LLVM_VERSION=`$LLVM_CONFIG --version | sed 's/svn.*//g'`
>-  LLVM_VERSION_INT=`echo $LLVM_VERSION | sed -e 
>'s/\([[0-9]]\)\.\([[0-9]]\)/\10\2/g'`
>+LLVM_VERSION=`$LLVM_CONFIG --version | sed 's/svn.*//g'`
>+LLVM_LDFLAGS=`$LLVM_CONFIG --ldflags`
>+LLVM_BINDIR=`$LLVM_CONFIG --bindir`
>+LLVM_CPPFLAGS=`strip_unwanted_llvm_flags "$LLVM_CONFIG --cppflags"`
>+LLVM_CFLAGS=$LLVM_CPPFLAGS   # CPPFLAGS seem to be sufficient
>+LLVM_CXXFLAGS=`strip_unwanted_llvm_flags "$LLVM_CONFIG --cxxflags"`
>+LLVM_INCLUDEDIR=`$LLVM_CONFIG --includedir`
>+LLVM_LIBDIR=`$LLVM_CONFIG --libdir`
>+
>+AC_COMPUTE_INT([LLVM_VERSION_MAJOR], [LLVM_VERSION_MAJOR],
>+[#include "${LLVM_INCLUDEDIR}/llvm/Config/llvm-config.h"])
>+AC_COMPUTE_INT([LLVM_VERSION_MINOR], [LLVM_VERSION_MINOR],
>+[#include "${LLVM_INCLUDEDIR}/llvm/Config/llvm-config.h"])
>+
>+if test "x${LLVM_VERSION_MAJOR}" != x; 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'`
>+fi
>+
> LLVM_COMPONENTS="engine bitwriter"
> if $LLVM_CONFIG --components | grep -q '\'; then
> LLVM_COMPONENTS="${LLVM_COMPONENTS} mcjit"
>@@ -1591,17 +1609,10 @@ if test "x$enable_gallium_llvm" = xyes; then
> LLVM_COMPONENTS="${LLVM_COMPONENTS} irreader"
> fi
> fi
>-  LLVM_LDFLAGS=`$LLVM_CONFIG --ldflags`
>-  LLVM_BINDIR=`$LLVM_CONFIG --bindir`
>-  LLVM_CPPFLAGS=`strip_unwanted_llvm_flags "$LLVM_CONFIG --cppflags"`
>-  LLVM_CFLAGS=$LLVM_CPPFLAGS   # CPPFLAGS seem to be sufficient
>-  LLVM_CXXFLAGS=`strip_unwanted_llvm_flags "$LLVM_CONFIG --cxxflags"`
>-  LLVM_INCLUDEDIR=`$LLVM_CONFIG --includedir`
>-  LLVM_LIBDIR=`$LLVM_CONFIG --libdir`
>-  DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT"
>-  MESA_LLVM=1
>-
>-  dnl Check for Clang interanl headers
>+DEFINES="${DEFINES} -DHAVE_LLVM=0x0$LLVM_VERSION_INT"
>+MESA_LLVM=1
>+
>+dnl Check for Clang internal headers
> if test "x$enable_opencl" = xyes; then
> if test "x$CLANG_LIBDIR" = x; then
> CLANG_LIBDIR=${LLVM_LIBDIR}
>@@ -1611,8 +1622,8 @@ if test "x$enable_gallium_llvm" = xyes; then
> AC_MSG_ERROR([Could not find clang internal header stddef.h 
> in $CLANG_RESOURCE_DIR Use --with-clang-libdir to specify the correct path to 
> the clang libraries.]))
> fi
> else
>-  MESA_LLVM=0
>-  LLVM_VERSION_INT=0
>+MESA_LLVM=0
>+LLVM_VERSION_INT=0
> fi
> else
> MESA_LLVM=0

-- 
You are receiving this mail because:
You are the assignee for the bug.
___
mesa-dev mailing list
mesa-dev@lists.freedesktop.org
http://lists.freedesktop.org/mailman/listinfo/mesa-dev